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

Document popover focus #12688

Merged
merged 4 commits into from
Mar 7, 2014
Merged

Document popover focus #12688

merged 4 commits into from
Mar 7, 2014

Conversation

mdo
Copy link
Member

@mdo mdo commented Feb 11, 2014

This documents #10140, but while adding I hit the bug in #11788 (trigger: 'focus' doesn't work in Safari and apparently Firefox). When that bug gets fixed, we can merge this in.

@mdo mdo added docs labels Feb 11, 2014
@mdo mdo added this to the v3.2.0 milestone Feb 11, 2014
@mdo mdo mentioned this pull request Feb 11, 2014
@cvrebert
Copy link
Collaborator

@mdo This PR has an unrelated "Update Glyphicons" commit in it.

@mdo
Copy link
Member Author

mdo commented Feb 11, 2014

Derp, resolved.

@@ -1160,8 +1162,11 @@ <h3 class="popover-title">Popover left</h3>

<h3>Live demo</h3>
<div class="bs-example" style="padding-bottom: 24px;">
<a href="#" class="btn btn-lg btn-danger" data-toggle="popover" title="A Title" data-content="And here's some amazing content. It's very engaging. right?" role="button">Click to toggle popover</a>
<button type="button" class="btn btn-lg btn-danger bs-docs-popover" data-toggle="popover" title="Popover title" data-content="And here's some amazing content. It's very engaging. Right?">Click to toggle popover</a>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wrong closing tag here.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, all the instances of this code have mismatched tags.

mdo added a commit that referenced this pull request Mar 7, 2014
@mdo mdo merged commit 864a18f into master Mar 7, 2014
@mdo mdo deleted the document_popover_focus branch March 7, 2014 09:31
@mdo mdo mentioned this pull request Mar 7, 2014
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