Page MenuHomePhabricator
Differential D18501

Add a selected button ui state
ClosedPublic
Actions

Authored by chad on Aug 30 2017, 4:54 PM.
Tags
None
Referenced Files
F18803310: D18501.diff
Fri, Oct 17, 10:39 PM
F18797802: D18501.id44437.diff
Fri, Oct 17, 5:29 AM
F18762602: D18501.id44433.diff
Mon, Oct 6, 7:47 PM
F18099354: D18501.id44437.diff
Aug 9 2025, 4:00 AM
F18098454: D18501.diff
Aug 8 2025, 8:16 PM
F17939790: D18501.id.diff
Jul 31 2025, 3:26 AM
F17805386: D18501.id.diff
Jul 25 2025, 12:31 PM
F17799433: D18501.diff
Jul 25 2025, 5:01 AM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Commits
rP11046d495dbf: Add a selected button ui state
Summary

Only for grey buttons, but can expand. Sets a selected class.

Test Plan

Review new changes in UIExamples.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad created this revision.Aug 30 2017, 4:54 PM
Herald added a subscriber: Korvin. · View Herald TranscriptAug 30 2017, 4:54 PM
Harbormaster completed remote builds in B18218: Diff 44433.Aug 30 2017, 4:55 PM
epriestley accepted this revision.Aug 30 2017, 4:56 PM
epriestley added inline comments.
webroot/rsrc/js/phuix/PHUIXButtonView.js
50–54

Simpler as:

this._selected = selected;
JX.DOM.alterClass(node, 'selected', this._selected);
This revision is now accepted and ready to land.Aug 30 2017, 4:56 PM
chad updated this revision to Diff 44435.Aug 30 2017, 5:01 PM
  • simpler
Harbormaster completed remote builds in B18220: Diff 44435.Aug 30 2017, 5:02 PM
Closed by commit rP11046d495dbf: Add a selected button ui state (authored by Chad Little <chad@phacility.com>). · Explain WhyAug 30 2017, 5:14 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
18 lines
src/
applications/
uiexample/
examples/
4 lines
view/
phui/
10 lines
webroot/
rsrc/
css/
phui/
button/
13 lines
js/
phuix/
8 lines

Diff 44437

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/uiexample/examples/PHUIButtonExample.php

Loading...
View Options

src/view/phui/PHUIButtonView.php

Loading...
View Options

webroot/rsrc/css/phui/button/phui-button.css

Loading...
View Options

webroot/rsrc/js/phuix/PHUIXButtonView.js

Loading...
Phabricator · Built by Phacility