Skip to content

Develop#733

Open
SerhiyDmytruk wants to merge 11 commits intomate-academy:masterfrom
SerhiyDmytruk:develop
Open

Develop#733
SerhiyDmytruk wants to merge 11 commits intomate-academy:masterfrom
SerhiyDmytruk:develop

Conversation

@SerhiyDmytruk
Copy link
Copy Markdown

No description provided.

- Updated stylelint from version 16.7.0 to 16.26.1 and added stylelint-config-standard.
- Modified typography styles to use Inter font for body text.
- Enhanced main.scss by importing additional modules and adjusting background color.
- Updated utility styles to include a new mobile-first container mixin and reset styles.
- Added new color variables for blue, black, and effect duration.
- Created new SVG icons for arrow, cookie, Facebook, Instagram, menu, phone, and close (X).
- Implemented new SCSS modules for bake, footer, header, hero, and info sections with responsive styles.
style: refactor SCSS variables and mixins for better maintainability

style: update color variables in main.scss and various module styles
Copy link
Copy Markdown

@2pasha 2pasha left a comment

Choose a reason for hiding this comment

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

good job! 👏

please add demo link to your PR description and please make request for mentor's review only when everything is ready

please take a look on these improvements:

  1. fix menu on tablet view
image
  1. add anchor links navigation for your nav links
image
  1. add hover effects for these elements
image
  1. add favicon to your page

  2. also add some hover effects for your images

image
  1. add some hover effects for your footer
image

@SerhiyDmytruk
Copy link
Copy Markdown
Author

Added style fix for elements in the screenshot.
thanks

@SerhiyDmytruk SerhiyDmytruk requested a review from 2pasha December 14, 2025 11:07
Copy link
Copy Markdown

@Anton-Kuchmasov Anton-Kuchmasov left a comment

Choose a reason for hiding this comment

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

Well done!

Don't forget to provide DEMO LINK in further tasks

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.

3 participants