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

Missing close icon and label of the modal in movie finder example #380

Closed
thomas-holzer opened this issue Mar 3, 2014 · 5 comments · Fixed by #440
Closed

Missing close icon and label of the modal in movie finder example #380

thomas-holzer opened this issue Mar 3, 2014 · 5 comments · Fixed by #440
Assignees
@connors
Labels
confirmed css
Milestone
2.0.2

Comments

@thomas-holzer
Copy link

Occurs on the second opening, please see the screenshot

missing close icon
@connors
Copy link
Collaborator

connors commented Mar 3, 2014

I'm having trouble reproducing this one. What browser/device are you using?

@connors connors added the css label Mar 3, 2014
@thomas-holzer
Copy link
Author

I can reproduce it with Chrome on Nexus 7 and on Samsung Galaxy S4 Mini.
I can also reproduce this issue using Chrome 33 on Windows 7 and emulating the touch events

@connors
Copy link
Collaborator

connors commented Mar 3, 2014

👍 Will check it out.

@connors connors added this to the 2.0.1 milestone Mar 3, 2014
@connors connors self-assigned this Mar 3, 2014
@mdo mdo modified the milestones: 2.0.2, 2.0.1 Mar 5, 2014
@connors connors mentioned this issue Mar 6, 2014
Merged
@wenzhixin
Copy link

0

1

2

Missing close icon and label of the modal with Chrome 33.0.1750.136 on Android 4.1.1.
But when I toggle the keyboard the icon and label is shown.

@connors
Copy link
Collaborator

connors commented Mar 12, 2014

@wenzhixin I believe this is still the same issue that was fixed in #440.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@connors connors

Labels
confirmed css
Projects
None yet
Milestone
2.0.2
Development

Successfully merging a pull request may close this issue.

Position .content absolutely twbs/ratchet
4 participants
@mdo @connors @thomas-holzer @wenzhixin