1368973 - Long page titles from web content loaded in the sidebar stretch the sidebar horizontally

Status: VERIFIED FIXED

(Firefox :: General, defect, P3)

Description

[Dão Gottwald [:dao]]

Attachment: screenshot

Comment 1

[Dão Gottwald [:dao]]

[Tracking Requested - why for this release]: See screenshot. This is quite broken.

Comment 2

[Marcia Knous [:marcia]]

Tracking 55+ per comment 1.

Comment 3

[David Bolter [:davidb] (NeedInfo me for attention)]

Marco is this regression expected to stay in 55 when it moves to beta? (Or is it caused by something behind a photon pref?)

Comment 4

[:Gijs (he/him)]

(In reply to David Bolter [:davidb] from comment #3)
> Marco is this regression expected to stay in 55 when it moves to beta? 

Yes, that's why it's tracked.

Comment 5

[:Gijs (he/him)]

I put up a patch on bug 1367909.

Comment 6

[:Gijs (he/him)]

This got fixed in bug 1367909.

Comment 7

[Dão Gottwald [:dao]]

Should get this verified specifically since bug 1367909 doesn't explicitly cover it.

Comment 8

[:Gijs (he/him)]

(In reply to Dão Gottwald [::dao] from comment #7)
> Should get this verified specifically since bug 1367909 doesn't explicitly
> cover it.

... and I specifically asked for that in bug 1367909. Please don't change the resolution. Whether this was fixed separately or with the same patch makes a difference for capacity / reserve maths, and so pretending it was fixed separately doesn't make sense.

Comment 9

[Dão Gottwald [:dao]]

(In reply to :Gijs from comment #8)
> (In reply to Dão Gottwald [::dao] from comment #7)
> > Should get this verified specifically since bug 1367909 doesn't explicitly
> > cover it.
> 
> ... and I specifically asked for that in bug 1367909.

Hadn't seen this. That's fine as long as it doesn't get lost, and ensuring this is easier in a separate bug.

> Please don't change
> the resolution. Whether this was fixed separately or with the same patch
> makes a difference for capacity / reserve maths, and so pretending it was
> fixed separately doesn't make sense.

*sigh* Bugzilla isn't just a place to upload patches, it's an issue tracker too. It helps us ship quality software. I wasn't pretending this was fixed separately, but insisting that it's a separate issue. We should fix whatever formula dictates that we cannot track this separately. For now, I'll just remove this from your photon radar to enable normal bugzilla and release tracking workflow.

Comment 10

[Dão Gottwald [:dao]]

o_O

Correct me if I'm wrong, but I believe qe‑verify is still used outside of the photon project. Or maybe I'm supposed to use verifyme here, not sure. Or maybe tracking+ already ensures QA attention? Adding the flag and keyword to be on the safe side.

Comment 11

[Camelia Badau [:cbadau], Desktop QA]

I've reproduced the issue on Nightly 55.0a1 (2017-05-31).
Verified fixed on Windows 7 x64, Ubuntu 14.04 x64 and Mac OSX 10.12.5 using Firefox 55 Beta 3 (buildID:  	20170619071723).