Skip to content

Ievgien iaroshenko #8

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

Draft
wants to merge 34 commits into
base: main
Choose a base branch
from
Draft

Ievgien iaroshenko #8

wants to merge 34 commits into from

Conversation

Ievgien
Copy link

@Ievgien Ievgien commented May 29, 2025

Draft PR.
Yet plan to work on

  1. Autherization storage.
  2. Validation messages
  3. Tests
  4. Error handling
  5. redirect wrong routes
  6. lazy load
  7. ratings
  8. add cache

border-radius: 14px;
box-shadow: 0 4px 24px rgba(78, 84, 200, 0.10);
padding: 2rem 2.5rem;
font-family: 'Segoe UI', Arial, sans-serif;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In terms of styling, on front-end we are usually trying to combine all fonts, colors, standard paddings, etc, to variables, colors common files.

You can put them in separate files, inside src/styles folder and then import them here using SCSS, e.g. @use "colors.scss" as colors

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you'll switch to CSS variables (like --main-padding: 8px), then you can even manipulate them though your code and make cool stuff, such as light/dark themes.

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