Allow action-list-items to also use text-overflow ellipsis
ClosedPublic
Actions

Authored by chad on Feb 6 2017, 5:24 PM.

Tags

None

Referenced Files

	F13168785: D17319.diff
	Tue, May 7, 7:56 AM

	Unknown Object (File)
	Mon, May 6, 1:09 PM

	Unknown Object (File)
	Fri, May 3, 7:39 AM

	Unknown Object (File)
	Fri, May 3, 5:18 AM

	Unknown Object (File)
	Thu, Apr 25, 1:39 AM

	Unknown Object (File)
	Thu, Apr 11, 9:28 AM

	Unknown Object (File)
	Wed, Apr 10, 5:15 PM

	Unknown Object (File)
	Tue, Apr 9, 3:18 AM

View All 64 Files

Subscribers

Korvin

Details

Reviewers

epriestley

Commits

rPb58e18bad785: Allow action-list-items to also use text-overflow ellipsis

Summary

These just cut off currently if the menu item name is too wide (like mobile). Add some ellipsis

Test Plan

Check long name on mobile action list menu

Diff Detail

Repository

rP Phabricator

Branch

mx-width-2 (branched from master)

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 15553
Build 20512: Run Core Tests
Build 20511: arc lint + arc unit

Event Timeline

chad created this revision.Feb 6 2017, 5:24 PM

Herald added a subscriber: Korvin. · View Herald TranscriptFeb 6 2017, 5:24 PM

epriestley accepted this revision.Feb 6 2017, 5:24 PM

This revision is now accepted and ready to land.Feb 6 2017, 5:24 PM

Harbormaster completed remote builds in B15553: Diff 41651.Feb 6 2017, 5:25 PM

Closed by commit rPb58e18bad785: Allow action-list-items to also use text-overflow ellipsis (authored by chad, committed by chad). · Explain WhyFeb 6 2017, 5:27 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

map.php

6 lines

webroot/

rsrc/

css/

phui/

phui-action-list.css

2 lines

Diff 41651

View Options

resources/celerity/map.php