Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • B bootstrap
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 263
    • Issues 263
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 114
    • Merge requests 114
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Bootstrap
  • bootstrap
  • Issues
  • #6835
Closed
Open
Issue created Feb 08, 2013 by Administrator@rootContributor

dropdown issue "too much recursion"

Created by: Yohn

I think when we took out support for the touchstart feature in the dropdowm menus, we missed something.. at the bottom of the dropdown plugin theres this..

$(document)
  .on('.dropdown-menu', function (e) { e.stopPropagation() })

shouldnt this line have an action in it? like click.dropdown-menu instead of just .dropdown-menu? whats this line for anyways?

I first found this problem a yesterday when I was getting a "too much recursion" error when working with @jschr's bootstrap-modal plugin and reported it there cause it didnt happen without jschrs plugin

my work arounds would be to either add the click event handler in that line, or simply take it out, both solutions work but I'm not sure exactly what its doing so I wouldnt take it out just yet..

guess I should show ya a fiddle to - http://jsfiddle.net/mh4hY/ fiddle works now since jschr updated his plugin with the code posted below

Assignee
Assign to
Time tracking