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.

Sign up for GitHub

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

Jump to bottom

Modals in Chrome 38 on Windows #14972

Closed
rhalaly opened this issue Nov 3, 2014 · 15 comments · Fixed by #14985
Closed

Modals in Chrome 38 on Windows #14972

rhalaly opened this issue Nov 3, 2014 · 15 comments · Fixed by #14985
Labels
browser-bug confirmed
Milestone
v3.3.1

Comments

@rhalaly
Copy link

rhalaly commented Nov 3, 2014

In the last update of Chrome (ver 38). The page can scroll while modal is open. "overflow:hidden" don't block the scrolling anymore.
@cvrebert
Copy link
Collaborator

cvrebert commented Nov 3, 2014

What OS are you using?
I can't seem to reproduce on Mac OS X with the demo modal in the docs.
Could you give a step-by-step description of how to reproduce the issue?

@cvrebert cvrebert added the css label Nov 3, 2014
@rhalaly
Copy link
Author

rhalaly commented Nov 3, 2014

I am using Chrome 38 in Windows (tested on Windows 7/8/8.1). Just open "small-modal" or "large-modal" demos in the docs and you will able scroll the page while modal is open.

@hnrch02 hnrch02 added this to the v3.3.1 milestone Nov 3, 2014
@hnrch02
Copy link
Collaborator

hnrch02 commented Nov 3, 2014

I can reproduce. Only on Windows though.

@cvrebert cvrebert changed the title Modals in Chrome 38 Modals in Chrome 38 on Windows Nov 3, 2014
@hnrch02
Copy link
Collaborator

hnrch02 commented Nov 3, 2014

This is not a regression on our side though. Just tried it with the 3.2 docs and it also happens there.

@rhalaly
Copy link
Author

rhalaly commented Nov 3, 2014

Its only happened when Chrome was updated to version 38.

@cvrebert
Copy link
Collaborator

cvrebert commented Nov 3, 2014

Could someone else please file the Chrome bug this time? 😒

@hnrch02
Copy link
Collaborator

hnrch02 commented Nov 3, 2014

I'll do it.

@hnrch02
Copy link
Collaborator

hnrch02 commented Nov 3, 2014

Filed an issue with Chromium: http://crbug.com/429604

@mkurz
Copy link

mkurz commented Nov 3, 2014

Can also reproduce with Chrome 38 on Ubuntu 12.04.

@hnrch02
Copy link
Collaborator

hnrch02 commented Nov 3, 2014

@mkurz Can you comment that on the Chromium bug please? Never mind, you just did 😄

@mkurz
Copy link

mkurz commented Nov 3, 2014

@hnrch02 Done.

@carasmo
Copy link

carasmo commented Nov 3, 2014

Can not reproduce with Chrome 38 Mac. Works great.

@cvrebert
Copy link
Collaborator

cvrebert commented Nov 3, 2014

@hnrch02 Add the bug to the Wall?

hnrch02 added a commit that referenced this issue Nov 3, 2014
@hnrch02
Add Wall of Browser Bugs entry for #14972
17a6b5e
@hnrch02 hnrch02 mentioned this issue Nov 3, 2014
Merged
hnrch02 added a commit that referenced this issue Nov 3, 2014
@hnrch02
Merge pull request #14985 from twbs/14972-wall
deab673 
Add Wall of Browser Bugs entry for #14972
@cvrebert
Copy link
Collaborator

cvrebert commented Nov 4, 2014

😍

@hnrch02 hnrch02 mentioned this issue Nov 4, 2014
Closed
@mkurz mkurz mentioned this issue Nov 12, 2014
Closed
@cvrebert cvrebert mentioned this issue Nov 17, 2014
Closed
cvrebert added a commit that referenced this issue Nov 20, 2014
@cvrebert
Update Wall of Browser Bugs entry for #14972
44735c8 
http://crbug.com/429604 has been merged into http://crbug.com/175502
cvrebert added a commit that referenced this issue Dec 2, 2014
@cvrebert
Update Wall of Browser Bugs entry for #14972
0445f23 
http://crbug.com/175502 is fixed in Chrome 40 and has been closed.
But Windows is still broken due to http://crbug.com/438064 ,
  which is open and currently targeting Chrome 41.
@juthilo juthilo mentioned this issue Dec 8, 2014
Closed
@cvrebert
Copy link
Collaborator

Per https://code.google.com/p/chromium/issues/detail?id=438064#c10 , this should now be fixed in Chrome Canary.

cvrebert added a commit that referenced this issue Dec 22, 2014
@cvrebert
http://crbug.com/438064 has been fixed
dc7ab34 
Refs #14972

[skip sauce]
[skip validator]
@cvrebert cvrebert mentioned this issue Dec 22, 2014
Merged
cvrebert added a commit that referenced this issue Dec 22, 2014
@cvrebert
Merge pull request #15424 from twbs/chrome-438064-fixed
bb9118f 
Remove #14972's entry from Wall of Browser bugs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
browser-bug confirmed
Projects
None yet
Milestone
v3.3.1
Development

Successfully merging a pull request may close this issue.

Add Wall of Browser Bugs entry for #14972 twbs/bootstrap
5 participants
@cvrebert @mkurz @hnrch02 @carasmo @rhalaly