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

fixed table headers wrong place on whole page scroll #4646

Closed
rcubetrac opened this issue Aug 19, 2014 · 7 comments
Closed

fixed table headers wrong place on whole page scroll #4646

rcubetrac opened this issue Aug 19, 2014 · 7 comments
Assignees
Milestone

Comments

@rcubetrac
Copy link

Reported by JohnDoh on 19 Aug 2014 07:05 UTC as Trac ticket #1490035

some one using one of my plugins reported an issue with the positioning of the fixed table headers in Chrome 36 and Opera 23. Please see johndoh/roundcube-sauserprefs#22

I was able to reproduce in Opera 23 on small screens. For example if I have a small screen (small enough to have a vertical scroll bar on the whole screen) and I'm viewing my mailbox. If I scroll the whole page down a bit then when I scroll the message list the fixed header bar is renendered lower down that it should be, not taking accoting of the overall page scroll.

Migrated-From: http://trac.roundcube.net/ticket/1490035

@rcubetrac
Copy link
Author

Comment by @thomascube on 19 Aug 2014 08:03 UTC

Page scrolling should not be possible in the views where we have fixed headers. That's a limitation of the feature as it uses position:fixed for keeping the header fixed.

@rcubetrac
Copy link
Author

Comment by @alecpl on 19 Aug 2014 13:07 UTC

I'm curious if position:sticky would be of any help here.

@rcubetrac
Copy link
Author

Comment by @alecpl on 6 Apr 2015 13:35 UTC

https://github.com/roundcube/roundcubemail/blob/master/program/js/list.js#L247

I'm curious why we do not call this code for webkit browsers. Removing the if fixes the issue in Chrome for me.

4ae28ff

@rcubetrac
Copy link
Author

Owner changed by @alecpl on 6 Apr 2015 13:35 UTC

=> thomasb

@rcubetrac
Copy link
Author

Milestone changed by @alecpl on 6 Apr 2015 13:35 UTC

later => 1.1.2

@rcubetrac
Copy link
Author

Comment by @alecpl on 31 Jul 2015 11:25 UTC

Fixed in 8b40575, let's see if it breaks anything.

@rcubetrac
Copy link
Author

Status changed by @alecpl on 31 Jul 2015 11:25 UTC

new => closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants