Page MenuHomePhabricator

Restrict all crumbs to 240 max width
ClosedPublic

Authored by chad on Feb 15 2017, 9:00 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 3 2024, 2:37 PM
Unknown Object (File)
Jan 26 2024, 9:39 AM
Unknown Object (File)
Dec 27 2023, 9:51 AM
Unknown Object (File)
Dec 27 2023, 9:51 AM
Unknown Object (File)
Dec 27 2023, 9:51 AM
Unknown Object (File)
Dec 21 2023, 4:25 PM
Unknown Object (File)
Dec 21 2023, 4:25 PM
Unknown Object (File)
Dec 21 2023, 4:25 PM
Subscribers

Details

Summary

Ref T12270. Any project, badge, dashboard, etc, that uses names in crumbs can over generate a long title. Restrict to a sane but generous width.

Test Plan

Make a project with a really long name, test various crumb layouts, boards, tasks, desktop, mobile.

Diff Detail

Repository
rP Phabricator
Branch
too-long-crumbs (branched from master)
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 15646
Build 20656: Run Core Tests
Build 20655: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Feb 15 2017, 9:03 PM
This revision was automatically updated to reflect the committed changes.

What browser did you say you had issues with this on? Safari? I can't reproduce - tried Safari, Chrome, Firefox on Mac:

pasted_file (1×2 px, 317 KB)

Safari, issue was with looking at user profiles where the crumb was short (like "alice").

(It would incorrectly render as "ali..." although there was plenty of room for it.)

yeah I can't reproduce that either. are you on an older Safari?

Seems dependent on the name, let me see if I can get you a better idea of what the repro case is.

Bad:

  • hector
  • joedecker
  • Joey
  • ducksey

Good:

  • joseph
  • alice
  • jsmith
  • sloth
  • pidgeon
  • omnibus
  • tortise-list
  • nnn
  • UPPERCASE
  • chrome
  • saurus

Uh no idea, maybe ending in "r" or "y"? Probably something about font metrics?

Yeah, I tried a dozen usernames, I can't reproduce it.

pasted_file (956×1 px, 182 KB)

oh ok, hector reproduces it.

"alice" is good, "alicy" is bad. "UPPERCASE" is good, "UPPERCASEy" is bad. So it depends on the last letter, I guess.