LiveGrid scrollbar disappears if scrollbar is too large in Firefox

[quodroc]

You can see it in the livegrid example http://www.sencha.com/examples/#livegrid
If you increase the size of the scrollbar beyond 18 pixels, either by increasing the DPI/font settings or manually increasing the scrollbar size, it disappears in Firefox.

Firefox will refuse to draw a scrollbar if the div is too narrow to fit it, and the livescroller div is given a fixed width of 18px from gxt-all.css.

Removing/overriding this CSS rule will allow the scrollbars to show up, but there is still a lesser problem:
the column widths don't line up with the scrollbar gutter.

Investigating further, there seems to be 2 reasons for this:

First, inside GridView, the scrollOffset field is set to a hardcoded 19 pixels, when it should really be Math.max(19, XDOM.getScrollBarWidth()) as seen in a few other places.

Changing this alone does not completely fix the problem, because XDOM.getScrollBarWidth() is actually broken in Firefox and always returns 0 if the scrollbar size exceeds 25 pixels.

[quodroc]

Also, the value '19' is hardcoded as the scrollbar size in a few other places,
(line numbers are for GXT 2.2.3)
GridView line 138 (mentioned above)
LiveGridView line 375
RowLayout lines 92, 189

[sven]

We will look into this for GXT3. There are serveral more changes needed to make this completly work.

I just did some fast testing for getScrollBarWidth and it works fine for me in FF.

[quodroc]

Just retested and for me, getScrollBarWidth works in Firefox as long as the width is less than 26px.

[quodroc]

For anyone interested, I have reached a satisfactory workaround by adding the following css:

.x-grid-panel .x-livegrid-scroller {
  width: auto;
  padding-left: 1px;
}

and overriding LiveGridView with

public LiveGridViewExtended() {
    super();
    scrollOffset = Math.max(19, XDOM.getScrollBarWidth());
}

[sven]

I had an outstanding commit that fixed this (including other things). I commited the scrollbar size calculation to SVN. I also removed the hardcoded size as you mentioned too.

[quodroc]

Thank you for the speedy resolution, much appreciated.