Skip to content

Fixed the responsive navigation menu #672

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

Merged
merged 1 commit into from
Oct 14, 2017
Merged

Fixed the responsive navigation menu #672

merged 1 commit into from
Oct 14, 2017

Conversation

javiereguiluz
Copy link
Member

This fixes #670. The responsive menu now it works as expected:

navbar

The only important change is this one:

// assets/js/app.js
+import 'bootstrap-sass/assets/javascripts/bootstrap/collapse.js';

The other changes were introduced by Webpack automatically.

@javiereguiluz javiereguiluz merged commit 18d8ec0 into symfony:master Oct 14, 2017
javiereguiluz added a commit that referenced this pull request Oct 14, 2017
This PR was merged into the master branch.

Discussion
----------

Fixed the responsive navigation menu

This fixes #670. The responsive menu now it works as expected:

![navbar](https://user-images.githubusercontent.com/73419/31537884-fac46ac0-b003-11e7-94d1-6fba6dc1ace7.gif)

The only important change is this one:

```diff
// assets/js/app.js
+import 'bootstrap-sass/assets/javascripts/bootstrap/collapse.js';
```

The other changes were introduced by Webpack automatically.

Commits
-------

18d8ec0 Fixed the responsive navigation menu
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.

button.navbar-toggle
2 participants