Skip to content

Always toggle the dropdowns when clicking on the toggle trigger#874

Closed
foca wants to merge 1 commit intotwbs:2.0-wipfrom
foca:patch-1
Closed

Always toggle the dropdowns when clicking on the toggle trigger#874
foca wants to merge 1 commit intotwbs:2.0-wipfrom
foca:patch-1

Conversation

@foca
Copy link
Copy Markdown

@foca foca commented Jan 2, 2012

Before this commit, whenever a dropdown menu is open and you click on the .dropdown-toggle link, it would remain open.

This commit will change that so the menu is closed by clicking again on the trigger.

Comment thread js/bootstrap-dropdown.js
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

why remove this?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Hm, now that you ask… I was only trying with a single drop down menu,
so I never noticed why that was there.

However, without removing that it won't hide the drop down when you
click on it while open. I'll look into it further tomorrow.

Cheers,

2012/1/2 Jacob Thornton
reply@reply.github.com:

@@ -41,9 +41,7 @@

       $parent.length || ($parent = $this.parent())

  •      clearMenus()

why remove this?


Reply to this email directly or view it on GitHub:
https://github.com/twitter/bootstrap/pull/874/files#r322803

@fat
Copy link
Copy Markdown
Member

fat commented Jan 7, 2012

feel free to reopen with correct code

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants