Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Scrollbar in drop-down list disappears after initial display #1390

Closed
Yanual opened this issue Jan 1, 2018 · 7 comments
Closed

Scrollbar in drop-down list disappears after initial display #1390

Yanual opened this issue Jan 1, 2018 · 7 comments
Labels
browser:chrome The issue can only be reproduced in the Chrome browser. resolution:upstream Issue in the third-party software.

Comments

@Yanual
Copy link

Yanual commented Jan 1, 2018

Issue is already reported on old issue tracking

see https://dev.ckeditor.com/ticket/14716

Reproduction steps

  1. Go to https://ckeditor.com/ckeditor-4 with chrome browser
  2. Click on the Styles drop-down in the Toolbar so the list of styles is displayed
  3. Click away so the list of styles is hidden
  4. Click on the Styles drop-down a second time - notice that when the list of styles is re-displayed the vertical scrollbar is now missing.

Expected result

Display Scrollbar each time the list is displayed

Actual result

After initial display the Scrollbar disappears

Other details

  • Browser: Chrome 62.0.3202.75
  • CKEditor version: 4.8.1

Additional Informations

This seems to be a chromium issue, see https://bugs.chromium.org/p/chromium/issues/detail?id=641881

Possible solution

For me the problem appears with the list of font sizes (I do not use styles and formats, incompatible with BBCode).
I made a change in panel / plugin.js on line 370 that works without disrupting the online menus and the color list.
Replace the code in line 370 (4.8.1 version) with this one :

// To avoid Chrome issue see https://bugs.chromium.org/p/chromium/issues/detail?id=641881
if ( CKEDITOR.env.chrome && this.attributes.role == 'listbox' && this.element.$.className == 'cke_panel_block' ) {
	this.element.setStyle( 'display', 'inline' );
} else {
	this.element.setStyle( 'display', '' );
}

Could you tell me if this is not awkward for other uses of CKEditor and if so it may help other people ?

@msamsel msamsel added status:confirmed An issue confirmed by the development team. type:bug A bug. labels Jan 5, 2018
@msamsel
Copy link
Contributor

msamsel commented Jan 5, 2018

I was able to confirm the issue.

@mlewand
Copy link
Contributor

mlewand commented Jan 5, 2018

@Yanual as you mentioned this is an upstream issue, and all we can do is hack to get a way around it. Please bump the original chromium issue (star it and/or put a comment) so that they see some interest in this issue.

@mlewand mlewand added the browser:chrome The issue can only be reproduced in the Chrome browser. label Jan 5, 2018
@Yanual
Copy link
Author

Yanual commented Jan 5, 2018

I just did it and I asked them why the status of the issue remains WontFix

@mlewand
Copy link
Contributor

mlewand commented Jan 5, 2018

They claimed that's no longer reproducible:

No longer reproduces in 60 or later.

That's why, though my bet is that the tester had used wrong environment where it's not reproducible. AFAIR it's not reproducible on mac.

@Yanual
Copy link
Author

Yanual commented Jan 5, 2018

Like I said I reproduce it on Browser: Chrome 62.0.3202.75 and on comment #16 he said -> I can confirm this issue happening in Windows and Linux in Chrome 63. It doesn't happen in MacOS with the "always show scrollbars" setting on.
That's why I'm perplexed about their analysis of feedback.

@mlewand
Copy link
Contributor

mlewand commented Jan 8, 2018

It looks like it's fixed on Chrome 65. We can already close this issue as it's an upstream. I'm happy to see it's fixed though. Version 65 is estimated to get stable at the beginning of March, so let's keep an eye on that.

@mlewand mlewand closed this as completed Jan 8, 2018
@mlewand mlewand added resolution:upstream Issue in the third-party software. and removed status:confirmed An issue confirmed by the development team. type:bug A bug. labels Jan 8, 2018
@williamsherlock
Copy link

williamsherlock commented Feb 23, 2018

This solution work for me:
Go to the ckeditor.js into callFunction and add this line before return.

$(".cke_combopanel").css("width", 'auto');

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
browser:chrome The issue can only be reproduced in the Chrome browser. resolution:upstream Issue in the third-party software.
Projects
None yet
Development

No branches or pull requests

4 participants