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

💄Update boilerplate generator theme #2047

Merged
merged 8 commits into from
May 13, 2019

Conversation

lluerich
Copy link
Collaborator

@lluerich lluerich commented May 9, 2019

fixes #1697

@lluerich lluerich requested a review from matthiasrohmer May 9, 2019 14:46
@lluerich lluerich changed the title Update/boilerplate generator theme 💄Update boilerplate generator theme May 9, 2019
@lluerich
Copy link
Collaborator Author

lluerich commented May 9, 2019

Forgot to add screenshots:
boilerplate-generator-1
boilerplate-generator-2

@matthiasrohmer matthiasrohmer requested review from sebastianbenz and removed request for matthiasrohmer May 9, 2019 15:32
Copy link
Collaborator

@sebastianbenz sebastianbenz left a 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:

  1. collapse the toggle when clicking on the first, currently selected, format
  2. 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
Copy link
Collaborator

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/.

Copy link
Collaborator

@sebastianbenz sebastianbenz May 9, 2019

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 :-).

Copy link
Collaborator Author

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? :-)

@lluerich
Copy link
Collaborator Author

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.

@sebastianbenz
Copy link
Collaborator

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

👍

Two other things I've noticed now that the hover is gone, can you please:

  1. increase the button height to 2rem
  2. increase the font-size inside the button to 16px

We also should take a look at the normal format switcher //cc @matthiasrohmer

@sebastianbenz
Copy link
Collaborator

Thanks! ...and I'm sorry, but I forgot two other things:

  1. Can you set the max-width for the format toggle to 240px
  2. the caret needs to reflect the collapsed state

@sebastianbenz
Copy link
Collaborator

Perfect! Thanks!

@sebastianbenz sebastianbenz merged commit 77cad07 into future May 13, 2019
@sebastianbenz sebastianbenz deleted the update/boilerplate-generator-theme branch May 13, 2019 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update Boilerplate Generator theme
3 participants