Opened 9 years ago

Closed 8 years ago

#6941 closed Bug (fixed)

horizontal scrollbar in firefox

Reported by: Holger Hees Owned by:
Priority: Normal Milestone:
Component: General Version: 3.5
Keywords: Firefox HasPatch Cc: Sa'ar Zac Elias


which only occurs in the initialization for 1 second to and is dependent on their parents' style

you need to change

_source/plugins/wysiwygarea/plugin.js line 537


element.setStyles( { position : 'absolute', top : '-3000px' } );


element.setStyles( { position : 'absolute', left: '-3000px', top : '-3000px' } );

Attachments (1)

test.html (638 bytes) - added by Holger Hees 9 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 Changed 9 years ago by Alfonso Martínez de Lizarrondo

Keywords: Firefox HasPatch added

comment:2 Changed 9 years ago by Garry Yao

Status: newpending

Could you provide a test case that shows the scrollbar?

Also, #5956 must be re-checked as there's no originate TC and I was unable to reproduce it in FF.

Changed 9 years ago by Holger Hees

Attachment: test.html added


comment:3 Changed 9 years ago by Holger Hees

the reason is the style 'overflow:auto' in the testcase.

and the patch was not complete. i have forget to save the original left value.

previousStyles = {

position : element.getStyle( 'position' ), left : element.getStyle( 'left' ) top : element.getStyle( 'top' )


element.setStyles( { position : 'absolute', left: '-3000px', top : '-3000px' } );

comment:4 Changed 9 years ago by Holger Hees

Is nobody interested on the bug?

comment:5 Changed 9 years ago by Holger Hees

what is wrong?

comment:6 Changed 9 years ago by Garry Yao

Status: pendingconfirmed

Regression of [5835], we need to verify the original TC at #5956.

comment:7 Changed 9 years ago by Garry Yao

Cc: Sa'ar Zac Elias added

comment:8 Changed 9 years ago by Sa'ar Zac Elias

We cannot use the left property because of our need for RTL compatibility. I'm not sure how much we can do in this sense to make it work well with overflowing parent, as also the regular way to check if right or left should be used will fail with them.

comment:9 Changed 9 years ago by Garry Yao

For me, #5956 is simply not reproducible (or at least it's missing a tc), even if it's valid, hidden/opacity fix should be considered in prior to an offset hack.

comment:10 Changed 8 years ago by Garry Yao

Resolution: fixed
Status: confirmedclosed

Expired on trunk.

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy