970897 - The History panel is too small

Status: VERIFIED FIXED

(Firefox :: Toolbars and Customization, defect)

Description

[u428464]

When the history widget is put in the toolbar the panel is very small and a scrollbar immediately appears. The panel should take more space on the screen before it shrinks and before a scrollbar appears.

Comment 1

[:Gijs (he/him)]

Backing out bug 966905 fixes this. Sigh.

Jared, want to have a look at what's going on here?

Guillaume, does this happen when it's in the menu? It doesn't for me, but I just want to doublecheck I'm not missing anything. Thanks!

Comment 2

[u428464]

(In reply to :Gijs Kruitbosch from comment #1)
> Guillaume, does this happen when it's in the menu? It doesn't for me, but I
> just want to doublecheck I'm not missing anything. Thanks!

No it doesn't happen as a subview. ;-)

Comment 3

[Jared Wein [:jaws] (please needinfo? me)]

I can't reproduce this. It was something that I noticed while working on bug 966905, but I was pretty sure I addressed it.

Possibly moving the -moz-box-flex:1 property from `.panel-subview-body` to `#PanelUI-popup .panel-subview-body` will fix it.

Comment 4

[:Gijs (he/him)]

Attachment: use subview inner scroll height for Australis standalone panels,

This fixes things for me.

Comment 5

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 8374182 [details] [diff] [review]
use subview inner scroll height for Australis standalone panels,

Review of attachment 8374182 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/components/customizableui/content/panelUI.js
@@ +302,5 @@
>                                   viewNode.querySelector(".panel-subview-footer"));
>  
>        let multiView = document.createElement("panelmultiview");
>        tempPanel.appendChild(multiView);
> +      multiView.setAttribute("mainviewissubview", "true");

such a hard string to read when joined without spaces :(

it may be against convention, but can this use camelCase or something that has more ascenders/descenders?

Comment 6

[:Gijs (he/him)]

switched to camel case:

remote:   https://hg.mozilla.org/integration/fx-team/rev/b301202a5eb7

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/b301202a5eb7

Comment 8

[u428464]

The panel isn't too small anymore but the scrollbar is always present which is still a bug.

Comment 9

[:Gijs (he/him)]

(In reply to Guillaume C. [:ge3k0s] from comment #8)
> The panel isn't too small anymore but the scrollbar is always present which
> is still a bug.

Please file a separate bug with steps to reproduce.

Comment 10

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 8374182 [details] [diff] [review]
use subview inner scroll height for Australis standalone panels,

[Approval Request Comment]
Bug caused by (feature/regressing bug #): regression from bug 966905
User impact if declined: subviews when used as panels in the toolbar will have incorrect height
Testing completed (on m-c, etc.): on m-c for a day now
Risk to taking this patch (and alternatives if risky): none
String or IDL/UUID changes made by this patch: none

note: needs to land on top of the patch for bug 966905.

Comment 11

[Jared Wein [:jaws] (please needinfo? me)]

https://hg.mozilla.org/releases/mozilla-aurora/rev/a41968fa2d02

Comment 12

[Gabriela [:gaby2300]]

Verified fixed using Windows 7 64 bit and Mozilla/5.0 (Windows NT 6.1; WOW64; rv:30.0) Gecko/20100101 Firefox/30.0. Verified in latest Nightly as well, setting the STATUS to VERIFIED.

Comment 13

[Ada [:adalucinet]]

I can also confirm this as verified fixed with latest Fx 30 beta 7 (Build ID: 20140522105902) on Windows 7, both 64bit and 32bit.