-
Notifications
You must be signed in to change notification settings - Fork 694
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
💄Update boilerplate generator theme #2047
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot for looking into this!
One thing: can we change the behavior of the format toggle:
- collapse the toggle when clicking on the first, currently selected, format
- collapse the toggle when selecting a different format
This is not a problem with the normal format selector as it always reloads the page.
@@ -0,0 +1,80 @@ | |||
/* | |||
Monokai style |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think a better place for this would be /frontend/scss/
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would indeed be a better place, but I forgot that the boilerplate generator uses it;s own build. So, please ignore and leave things as they are :-).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So should I revert the changes completely or just leave them in the state I submitted yesterday? :-)
I changed the behaviour of the dropdown to only open on click, because I saw no way to have both the hover effect and let the dropdown collapse once something is selected or the selected item is clicked. |
👍 Two other things I've noticed now that the hover is gone, can you please:
We also should take a look at the normal format switcher //cc @matthiasrohmer |
Thanks! ...and I'm sorry, but I forgot two other things:
|
Perfect! Thanks! |
fixes #1697