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

Change of flipdown.css styles to CSS variables #36

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

alexvishweb
Copy link

Hello,

thank you for the great plugin!

I've used it in my latest project and I liked it but restyling wasn't very comfortable. Had to change same values in many places + change them for different screen sizes, etc. That's a lot of work, so I rewrote flipdown.css styles using CSS variables and now all styles can be changed in one place. The whole look can be altered with a couple of variables. I also removed media query rules from CSS since every project has its own media query breakpoints and there is no more need to show which rules should be changed with media queries since the size is now changed with one variable.

For example,

@media (max-width: 550px) {
  .flipdown {
    --rotor-base: 50px;
  }
}

will make flipdown 2 times smaller (initial --rotor-base is equal to 100px for simpler calculations)

Other sizes are changed with multipliers.
For example, widths of rotors can be changed by changing "0.5" in a variable --rotor-width: calc(var(--rotor-base) * 0.5);. 100px * 0.5 = 50px.

The colors can be changed pretty easy too by altering these variables

--flip-base: 0, 0%;
--flip-l: 0%;

that represent HSL (Hue, Saturation, Lightness) model
https://css-tricks.com/hsl-hsla-is-great-for-programmatic-color-control/

Feel free to ask any questions!

@mariusa
Copy link

mariusa commented Feb 7, 2023

@PButcher Peter, are you still around? Could this be merged, please? Thanks

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.

2 participants