Page MenuHomePhabricator

[web] Add ellipsis on longer thread names
ClosedPublic

Authored by angelika on Mon, Oct 14, 8:19 AM.
Tags
None
Referenced Files
F2983617: D13706.id45217.diff
Wed, Oct 16, 9:20 AM
F2983015: D13706.id45171.diff
Wed, Oct 16, 7:20 AM
F2982460: D13706.diff
Wed, Oct 16, 3:36 AM
Unknown Object (File)
Tue, Oct 15, 4:33 AM
F2974409: Screenshot 2024-10-15 at 13.22.51.png
Tue, Oct 15, 4:24 AM
F2974406: Screenshot 2024-10-15 at 13.21.08.png
Tue, Oct 15, 4:24 AM
Unknown Object (File)
Tue, Oct 15, 3:17 AM
Unknown Object (File)
Tue, Oct 15, 3:15 AM
Subscribers

Details

Summary

This diff adds ellipsis to longer thread names

Before:

Screenshot 2024-10-15 at 13.21.08.png (766×2 px, 180 KB)

After:

Screenshot 2024-10-15 at 13.22.51.png (642×2 px, 131 KB)

Test Plan
  1. Create a community with a longer name
  2. Verify that the ellipsis is added when thread name doesn't fit in browser window

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

angelika edited the summary of this revision. (Show Details)

nit: worth adding a link to the Linear issue in diff summary, this always gives a reviewer a bit more context

This revision is now accepted and ready to land.Tue, Oct 15, 11:38 PM