[Webkit-unassigned] [Bug 69993] CSS2 overflow: scrollbar not visible on SELECT elements when overflow: scroll is set

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=69993





--- Comment #11 from Antaryami Pandia <xqb748 at motorola.com>  2011-10-18 22:44:26 PST ---
(In reply to comment #10)
> > For overflow:scroll, currently webkit always places the horizontal and vertical scrollbar.Related code can be found at RenderBlock::layoutBlock.
> 
> Yes, but why make an exception just for <select>? What you see is that overflow:scroll scrollbars are positioned so that they obscure <select>'s own scrollbar, which likely happens because its bounds are computed incorrectly.
> 
> If you think that it's incorrect to always draw scrollbars for overflow:scroll, you could attack that issue in general.

Firefox and IE shows only vertical scrollbar for list box (<select> with size) with overflow:scroll.I think they also overlook the overflow:scroll property for this case. With this change webkit matches the other browser's behavior.
Also there is piece of of code present in CSSStyleSelector::adjustRenderStyle which always set menulist elements overflow as visible.

    // Menulists should have visible overflow
    if (style->appearance() == MenulistPart) {
        style->setOverflowX(OVISIBLE);
        style->setOverflowY(OVISIBLE);
    }

> > I have added test cases for overflow:scroll (select-overflow-scroll.html) and inherited case (select-overflow-scroll-inherited.html) and theirs pixel tests.
> > Were you talking about the overflow:scroll and inherited cases.Please let me know if I have misunderstood.
> 
> I didn't notice that you added another test, but there is still no test for getComputedStyle(). It should be tested, and it should work the same way it does in other browsers.

OK.I will add a test case for getComputedStyle. I checked the computed style (using firebug-inspect element) and the elements overflow can be seen as visisble, which matches with this change.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.