This adds minor functionality to PHUIHeaderView (icons, tags) that were previously on in ActionHeader. This allows us to remove the PHUIActionHeaderView outright in favor of just one Header class.
Details
Details
- Reviewers
epriestley btrahan - Maniphest Tasks
- T8099: Mid-2015 Phabricator Redesign
- Commits
- Restricted Diffusion Commit
rPe5920618912a: [Redesign] Remove PHUIActionHeaderView
Tested each callsite, workboards, hovercards, conpherence.
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
src/applications/conpherence/controller/ConpherenceWidgetController.php | ||
---|---|---|
71–86 | This section doesn't work, the JS fails and I'm not adept here enough to know what I need to tweak. The JS that fails is: var widget_header = JX.DOM.find(widget_pane, 'a', 'widgets-selector'); |
src/applications/conpherence/controller/ConpherenceWidgetController.php | ||
---|---|---|
71–86 | Maybe just leave a comment and we can clean it up later before merging? Offhand, it might just be widgets-selector vs widgets_selector (hyphen vs underscore), as written. |
src/applications/conpherence/controller/ConpherenceWidgetController.php | ||
---|---|---|
71–86 |