1328519 - Redefine the width of downloadDetails.width in the L10N file (downloads.dtd) for Downloads Panel

Status: RESOLVED FIXED

(Firefox :: Downloads Panel, defect, P1)

Description

[Sean Lee [:seanlee][:weilonge]]

The width of downloadDetails.width in the L10N file (downloads.dtd) should be refined to be fulfill the latest design of Downloads Panel including:
1. The original downloadDetails string.
2. The string when hover on an item.
3. The string when hover on an action button.

(In reply to :Paolo Amadini from comment #48)
> Comment on attachment 8817589 [details]
> Bug 812894 - Use the short time format for downloading remaining time.;
> 
> https://reviewboard.mozilla.org/r/97820/#review98604
> 
> ::: browser/locales/en-US/chrome/browser/downloads/downloads.dtd:26
> (Diff revision 2)
> >  
> > -     59 minutes, 59 seconds remaining - 1022 of 1023 KB
> > +     59m 59s left - 1022 of 1023 KB (120.5 KB/sec)
> >  
> > -     That's 50 characters, so we set the width at 50ch.
> > +     That's 45 characters, so we set the width at 45ch.
> >       -->
> > -<!ENTITY downloadDetails.width            "50ch">
> > +<!ENTITY downloadDetails.width            "45ch">
> 
> We should keep the current value even if the string for in-progress
> downloads is shorter, because the strings for blocked downloads and the new
> labels we are adding for hover cases may be longer.
> 
> In fact, the way the rule is explained in the comment is not that useful. We
> can update the comment for now to explain, but we may actually consider
> redefining these widths in a separate bug at some point.
>

Comment 1

[:Paolo Amadini]

When doing this, we can simplify how this width is defined. Instead of changing the font size on the entire item and reverting it for the file name, we can just make the font size smaller for the details label. In fact, if I understand correctly, the current solution was implemented to keep this width unchanged and keep the result slightly different on various platforms, which is probably unnecessary.

Comment 2

[:Paolo Amadini]

Attachment: Bug 1328519 - Simplify the definition of the Downloads Panel width. r=mak

The width is now defined on the entire list instead of the inner elements, similarly to other Photon panels like the main menu. This is a prerequisite for preventing layout issues while removing the "download" binding.

Comment 3

[Marco Bonardo [:mak]]

Comment on attachment 9010351 [details]
Bug 1328519 - Simplify the definition of the Downloads Panel width. r=mak

Marco Bonardo [::mak] has approved the revision.

Comment 4

[:Paolo Amadini]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=ac5cd95875d7174da558b184febfa685f62da13d

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/83ab7c05f538
Simplify the definition of the Downloads Panel width. r=mak

Comment 6

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/83ab7c05f538