Page MenuHomePhabricator

Fix cursor on mock card hover in pholio
ClosedPublic

Authored by lpriestley on May 5 2014, 11:30 PM.
Tags
None
Referenced Files
F13255894: D8986.id21337.diff
Sat, May 25, 9:07 AM
F13253275: D8986.diff
Sat, May 25, 2:33 AM
F13241748: D8986.id21336.diff
Wed, May 22, 11:12 PM
F13241747: D8986.id21335.diff
Wed, May 22, 11:12 PM
F13241745: D8986.id21334.diff
Wed, May 22, 11:12 PM
F13241743: D8986.id.diff
Wed, May 22, 11:12 PM
F13241740: D8986.diff
Wed, May 22, 11:12 PM
F13236885: D8986.diff
Tue, May 21, 11:22 AM
Subscribers

Details

Summary

Fixes T4888, make seemingly actionable area in Pholio "Mocks List" actually actionable

Test Plan

Open pholio, create a mock, return to list of all mocks, hover over mock, only image in mock should switch to pointer on hover.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

lpriestley retitled this revision from to Fix cursor on mock card hover in pholio.
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added a reviewer: epriestley.

Can we just remove both these rules completely? The image link should get a pointer because it's inside a link, and the default cursor isn't overriding anything that I can see.

lpriestley edited edge metadata.

Removing some cursor rules

epriestley edited edge metadata.
This revision is now accepted and ready to land.May 6 2014, 12:06 AM
epriestley updated this revision to Diff 21337.

Closed by commit rPb6185314512e (authored by @lpriestley, committed by @epriestley).