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

fix(menu): add width and small-width css variables #14997

Merged

Conversation

adamlacombe
Copy link
Contributor

Short description of what this resolves:

Allow the menu width to be modified using css variables.

Changes proposed in this pull request:

  • add --width css variable to menu
  • add --small-width css variable to menu

Ionic Version: 4.0.0-beta.1

@ionitron-bot ionitron-bot bot added the package: core @ionic/core package label Aug 2, 2018
@manucorporat
Copy link
Contributor

@adamlacombe Hi! thanks a lot for the PR, one small request, can we use --width-small, we would like both variables to be grouped

@adamlacombe adamlacombe force-pushed the menu-add-css-variable-width branch from c11e1bb to 22e7e95 Compare August 2, 2018 18:40
@adamlacombe
Copy link
Contributor Author

Sure, not a problem. Just pushed the requested changes. Thanks!

@manucorporat manucorporat merged commit 69f5cc8 into ionic-team:master Aug 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: core @ionic/core package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants