Themes with max_length unbroken name are causing a large size installation popup

NEW
Unassigned

Status

()

Toolkit
Add-ons Manager
P5
normal
25 days ago
19 days ago

People

(Reporter: ValentinaP, Unassigned)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(firefox56 affected, firefox57 wontfix, firefox58 affected)

Details

Attachments

(1 attachment)

(Reporter)

Description

25 days ago
Created attachment 8923752 [details]
LONG_NAME.png

[Affected versions]:
Firefox 56.0.2 (64-bit)
Firefox 58.0a1 (2017-10-30) (64-bit)

[Affected platforms]:
Windows 7 64-bit
Mac OS X 10.13

[Steps to reproduce]:
1. Launch Firefox with clean profile.
2. Navigate to https://addons-dev.allizom.org
3. Choose a theme with unbroken name (50 chars lenght) i.e.https://addons-dev.allizom.org/en-US/firefox/addon/testthemetestthemetestt/
4. Install that theme 
5. Observe confirmation popup

[Expected Results]:
There are no display or layout issues.

[Actual Results]:
Theme name inside confirmation popup is not wrapped and therefor is causing layout issues.

[Additional Notes]:
 - NOT reproducible for extensions
Attaching screenshot for this issue.
I think the way that showed up isn't too bad.  But Markus, how should we be handling long names?
Flags: needinfo?(mjaritz)
OS: Windows 7 → All
Hardware: x86_64 → All

Comment 2

24 days ago
I doubt we should be worrying about this one and focusing on the redesign instead.

Comment 3

24 days ago
Please ignore comment 2, I thought this bug was about something different. There is no redesign in the works. Honest.

Updated

23 days ago
status-firefox57: --- → wontfix
Priority: -- → P5
(In reply to Andrew Swan [:aswan] from comment #1)
> I think the way that showed up isn't too bad.  But Markus, how should we be
> handling long names?

The popups should not change width, but break the long name into multiple lines.
(like css word-wrap: break-word;)
Flags: needinfo?(mjaritz)
You need to log in before you can comment on or make changes to this bug.