On 6/10/09, P T Withington <[email protected]> wrote: > > I'm _really_ concerned about tweaking the padding without understanding > why. In particular, you may end up clipping the text in other browsers > where the clipping already works. > > Things I would investigate: > > 1) Did IE change the size of its scrollbars? Is that why they are sticking > out now? (Currently the scrollbar width is browser-specific, because IE > _used_ to have smaller ones than the other browsers.
It didn't look like the IE scrollbars were partially clipped, it looked like they were not being clipped at all. This is in contrast to the appearance in Firefox or Safari, where they did not appear at all. 2) Why do the scrollbars _not_ stick out the same way when you are not > moused over? There must be some additional difference between the _click > and non-_click divs. Yeah the mystery is that it seems like if the container click-div didn't have overflow=hidden set, that the native scrollbars ought to be showing up in all browsers. So I guess the question is not 'why did they appear in IE?" but "why didn't they appear in Firefox/Safari?" It's _really_ important to understand what is going on here and not just > fudge things by a few pixels to make them work. This stuff is all very > delicate and if you stick in a fudge factor for one browser that is not > controlled by a quirk, you are liable to break all the others. Well, I tested the scrolledittext in all the browsers, but I guess I had better try to understand the reason why the scrollbars were being hidden in Firefox and Safari. On 2009-06-10, at 15:44EDT, Henry Minsky wrote: > > Change 20090610-hqm-t by [email protected] on 2009-06-10 15:42:36 EDT >> in /Users/hqm/openlaszlo/trunk-diamond >> for http://svn.openlaszlo.org/openlaszlo/trunk >> >> Summary: fix for DHTML - IE7: scrolledittext with custom vertical >> scrollbar shows scrollbars incorrectly >> >> New Features: >> >> Bugs Fixed: LPP-8257 >> >> Technical Reviewer: max >> QA Reviewer: ptw >> Doc Reviewer: (pending) >> >> Documentation: >> >> Release Notes: >> >> Details: >> >> + modified the container div, lzinputtextmultilinecontainer_click, to have >> CSS overflow=hidden. >> I also needed to make it's padding 2 pixels smaller, or else some of the >> native scrollbars still showed >> through. >> >> >> Tests: >> >> test case in bug report now works without native scrollbars appearing >> >> Files: >> M WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js >> >> Changeset: >> http://svn.openlaszlo.org/openlaszlo/patches/20090610-hqm-t.tar >> > > -- Henry Minsky Software Architect [email protected]
