Home GnuPG

Fix property shadowing on ActionListView after conversion to TagView
90258ed491d5Unpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

Fix property shadowing on ActionListView after conversion to TagView

Summary:
See D17222. D17209 accidentally broke setting IDs on ActionListView by converting it into a TagView: TagView already has an id property, and this new id property on the subclass shadows it.

Materially, the "Actions" mobile button in the headers of objects (for example: Maniphest Task -> shrink browser window -> click "Actions" next to task name) relies on setting IDs on list views.

Test Plan:

  • Viewed a task.
  • Made browser window narrow.
  • Clicked [= Actions] button.
  • After patch: saw a dropdown menu.

Reviewers: chad

Reviewed By: chad

Differential Revision: https://secure.phabricator.com/D17223

Details

Provenance
epriestley <git@epriestley.com>Authored on Jan 18 2017, 8:29 PM
Parents
rPHAB35f4514e3ff6: Fancier user menu
Branches
Unknown
Tags
Unknown

Event Timeline