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

Fix scroll in Edge browser #5750

Closed
wants to merge 1 commit into from
Closed

Fix scroll in Edge browser #5750

wants to merge 1 commit into from

Conversation

peterkroon
Copy link

I was unable to scroll the left pane.
With the z-index set to 4 the expected behavior is achieved.

Edge:
"Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.79 Safari/537.36 Edge/14.14393"

I was unable to scroll the left pane.
With the z-index set to 4 the expected behavior is achieved.

Edge:
"Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.79 Safari/537.36 Edge/14.14393"
@alecpl
Copy link
Member

alecpl commented May 5, 2017

Works for me with the same Edge version. What Roundcube version? What plugins? What skin?

@peterkroon
Copy link
Author

I have a default installation with the filters plugin installed.
Skin is Larry.

Roundcube Webmail 1.2.4

Copyright © 2005-2016, The Roundcube Dev Team

This program is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by the Free Software Foundation, either version 3 of the License, or (at your option) any later version.
Some exceptions for skins & plugins apply.

Geïnstalleerde plug-ins

Invoegtoepassing	Versie	Licentie	Broncode
archive	2.4	GPLv3+	
filesystem_attachments	1.0	GPLv3+	
filters	2.1.6	GNU GPLv3+	Download
jqueryui	1.10.4	GPLv3+	
password	4.1	GPLv3+	
zipdownload	3.1	GPLv3+	``

@alecpl
Copy link
Member

alecpl commented May 5, 2017

Ok. In 1.3 I confirm the issue in desktop and list mode, but not in wide screen mode.

@alecpl alecpl added this to the 1.3.0 milestone May 5, 2017
@peterkroon
Copy link
Author

Are list mode and widescreen a 1.3 feature?
Can I reproduce this with 1.2.4 ?

@alecpl
Copy link
Member

alecpl commented May 5, 2017

Yes. It's an issue in 1.2 too. I'm preparing a fix with some small z-index cleanup.

@alecpl
Copy link
Member

alecpl commented May 5, 2017

Fixed.

@alecpl alecpl closed this May 5, 2017
@peterkroon peterkroon deleted the patch-1 branch May 6, 2017 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants