Skip to content

Develop#985

Open
OwlDevcUA wants to merge 8 commits into
mate-academy:masterfrom
OwlDevcUA:develop
Open

Develop#985
OwlDevcUA wants to merge 8 commits into
mate-academy:masterfrom
OwlDevcUA:develop

Conversation

@OwlDevcUA
Copy link
Copy Markdown

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.

Good work!

To improve:

  1. This card shouldn't stretch - even if there is only 1 card in Favourites section at the moment:
Image
  1. 16 items on page looks like only 4. Would you mind to fix this?
Image
  1. Add links to your Github, Linkedin etc. at the Footer navlinks

@OwlDevcUA OwlDevcUA requested a review from Anton-Kuchmasov May 2, 2026 16:09
Copy link
Copy Markdown

@etojeDenys etojeDenys left a comment

Choose a reason for hiding this comment

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

almost done

  1. products in this section should not be discounted
Image
  1. it is better to open these links in a new tab
Image
  1. make sure you are increasing these numbers as well
Image

@OwlDevcUA OwlDevcUA requested a review from etojeDenys May 5, 2026 13:34
Copy link
Copy Markdown

@etojeDenys etojeDenys 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

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