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

Resolution to Issue #20, Partial Resolution to Issue #4 and new toggle functionality #21

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jodyfarnden
Copy link

This version adds the ability to add custom svg icons to the buttons instead of being restricted to material icons. This should assist in resolution of #4. To use a custom icon it must be included in the project assets and the path and a custom name must be supplied to the button (parent menu button or child buttons will work)

The lowercase text css has been split into it's own class and a "toLower" input has been supplied to add the lowercase text styling if required. by default this is false and text appears normal. This should resolve #20.

An additional input "allowToggle" has been added - this allows the automatic retraction of the menu to be stopped if required. An example is if you are using the menu for an edit/save scenario, the save button press will not Toggle the menu if you set the allowToggle to false when validation fails.

Note that I have been unable to build the original project, I am testing the modified code in a new Angular 6 project and has been working, but may want to verify it still builds for the targeted version of angular.

Made toLower an optional text style
Added "allowToggle" input to control if menu autoToggles
allowToggle enables a save button to report errors
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Text labels should not be made lowercase
1 participant