Page MenuHomePhabricator

Clean up some PHUI/PHUIX button behaviors
ClosedPublic

Authored by epriestley on Jun 2 2017, 11:12 AM.
Tags
None
Referenced Files
F18770346: D18065.id43451.diff
Wed, Oct 8, 12:05 PM
F18694303: D18065.diff
Sat, Sep 27, 3:26 AM
F18621052: D18065.id.diff
Mon, Sep 15, 6:57 AM
F18615638: D18065.diff
Sun, Sep 14, 3:07 PM
F18508895: D18065.id.diff
Sep 5 2025, 3:05 AM
F18501189: D18065.diff
Sep 4 2025, 9:35 PM
F18418817: D18065.id43451.diff
Aug 30 2025, 1:53 PM
F18292563: D18065.id43454.diff
Aug 23 2025, 12:21 PM
Subscribers
None

Details

Summary

Ref T12733. Some minor issues:

  • The strlen(...) test against $this->text fails if a caller does something like setText(array(...)). This is rare, but used in DiffusionBrowseController, from D15487.
  • Add PHUIX examples for icon-only buttons.
  • Remove unused SIMPLE constant now that no callsites remain.
Test Plan
  • Viewed a directory in Diffusion's "Browse" view in a Git repository, no longer saw a warning / error log.
  • Viewed PHUIX Components UI examples.
  • Grepped for ::SIMPLE.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable