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

Feat/refactor css custom props and spelling #29

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

krypton225
Copy link

This PR makes some changes as following:

  1. Enhance all CSS properties with using custom CSS properties (or CSS variables) to improve flexibility if we need to change any thing easily.

  2. Make styles for colors and backgrounds in two prefers color scheme modes: light and dark to make the website more better with user preferences.

  3. Enhance some selector to use logical CSS properties like: margin-block-end, … etc to make the website with more flexibility.

  4. Update the mistake spelling in the text.

  5. Make font sizes as CSS custom properties (or CSS variables) to use it in hole website to improve responsive design.

  6. Use local scope CSS custom properties (or CSS variables) in some selectors.

Add colors for main one and others in `root` as `CSS variables` or `CSS custom properties`. We also added these variables in light mode and dark one for user prefer color scheme.
Update all `color` and `background-color` properties with `CSS variables` or `CSS custom properties`.
Add custom properties in CSS selectors and use inner custom properties to reflect with all properties which have vendor prefixes.
Copy link

vercel bot commented Jun 29, 2024

@krypton225 is attempting to deploy a commit to the ahmed0saber's projects Team on Vercel.

A member of the Team first needs to authorize it.

@ahmed0saber
Copy link
Owner

Hello @krypton225 , I have reviewed the great work you have done and we are about to merge it after fixing these small issues.
Great job, and thanks for your interest contributing to ahmed0saber/quran_mailer.

Best wishes!

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