Page MenuHomePhabricator
Differential D17931

Don't render an action list without actions
ClosedPublic
Actions

Authored by chad on May 17 2017, 4:07 PM.
Tags
None
Referenced Files
F18841881: D17931.id.diff
Tue, Oct 28, 11:59 AM
F18780863: D17931.id43130.diff
Sun, Oct 12, 8:20 PM
F18775205: D17931.id.diff
Fri, Oct 10, 6:16 PM
F18732586: D17931.id.diff
Tue, Sep 30, 6:53 PM
F18715252: D17931.diff
Sep 29 2025, 9:44 AM
F18595889: D17931.diff
Sep 12 2025, 11:06 PM
F18542806: D17931.diff
Sep 7 2025, 11:02 PM
F18401533: D17931.id.diff
Aug 29 2025, 7:24 PM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Commits
rP75fb1a0327cb: Don't render an action list without actions
Summary

Skips rendering of partial elements if no actions are present.

Test Plan

Tested on profile menu item page, maniphest curtain, phriction dropdown, and instance backups page (no actions at all).

Diff Detail

Repository
rP Phabricator
Branch
action-list (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 17054
Build 22807: Run Core Tests
Build 22806: arc lint + arc unit

Revision Contents
Changeset List

PathSize
resources/
celerity/
6 lines
src/
view/
layout/
4 lines
webroot/
rsrc/
css/
phui/
7 lines

Diff 43129

View Options

resources/celerity/map.php

Loading...
View Options

src/view/layout/PhabricatorActionListView.php

Loading...
View Options

webroot/rsrc/css/phui/phui-curtain-view.css

Loading...
Phabricator · Built by Phacility