Page MenuHomePhorge
Differential D25922

Fix invalid CSS cursor value for .phabricator-main-menu-brand
ClosedPublic
Actions

Authored by aklapper on Mon, Mar 24, 17:51.
Tags
None
Referenced Files
F3337385: D25922.1743588568.diff
Tue, Apr 1, 10:09
F3333535: D25922.1743536596.diff
Mon, Mar 31, 19:43
F3320369: D25922.1743328526.diff
Sat, Mar 29, 09:55
F3316524: D25922.1743252289.diff
Fri, Mar 28, 12:44
F3312526: D25922.1743211774.diff
Fri, Mar 28, 01:29
F3310522: D25922.1743195300.diff
Thu, Mar 27, 20:55
F3308408: D25922.1743165435.diff
Thu, Mar 27, 12:37
F3307847: D25922.1743153023.diff
Thu, Mar 27, 09:10
View All 9 Files
Subscribers
Cigaryno
Matthew
tobiaswiese
valerio.bozzolan

Details

Reviewers
valerio.bozzolan
Group Reviewers
O1: Blessed Committers
Commits
rPfede9f450555: Fix invalid CSS cursor value for .phabricator-main-menu-brand
Summary

The W3C CSS validator throws Value Error : cursor hand is not a cursor value : hand.
hand is not a valid value for cursor; likely pointer was meant.
See https://developer.mozilla.org/en-US/docs/Web/CSS/cursor

Test Plan

Paste main-menu-view.css into https://jigsaw.w3.org/css-validator/ and read some CSS docs.

Diff Detail

Repository
rP Phorge
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

aklapper created this revision.Mon, Mar 24, 17:51
Owners added a reviewer: O1: Blessed Committers.Mon, Mar 24, 17:51
Herald added subscribers: Cigaryno, Matthew, valerio.bozzolan, tobiaswiese. · View Herald TranscriptMon, Mar 24, 17:51

Thanks for this patch! Kind reminder: if you touched CSS or JavaScript, please remember to also run this:

./bin/celerity map

⚙️ Thanks! Bip bop I'm a bot - helping for T15209

aklapper requested review of this revision.Mon, Mar 24, 17:51
Harbormaster completed remote builds in B1802: Diff 2771.Mon, Mar 24, 17:51
valerio.bozzolan accepted this revision.Tue, Mar 25, 08:19

sgtm

This revision is now accepted and ready to land.Tue, Mar 25, 08:19
Closed by commit rPfede9f450555: Fix invalid CSS cursor value for .phabricator-main-menu-brand. · Explain WhyTue, Mar 25, 09:25
This revision was automatically updated to reflect the committed changes.
aklapper added a commit: rPfede9f450555: Fix invalid CSS cursor value for .phabricator-main-menu-brand.
aklapper mentioned this in Next Up.Tue, Mar 25, 09:41

Revision Contents
Changeset List

PathSizePackages
resources/
celerity/
O1: Blessed Committers
webroot/
rsrc/
css/
application/
base/
2 linesO1: Blessed Committers

Diff 2773

View Options

resources/celerity/map.php

Loading...
View Options

webroot/rsrc/css/application/base/main-menu-view.css

Loading...
Content licensed under Creative Commons Attribution-ShareAlike 4.0 (CC-BY-SA) unless otherwise noted; code licensed under Apache 2.0 or other open source licenses. · CC BY-SA 4.0 · Apache 2.0