Skip to content

add solution#991

Open
dimon2202 wants to merge 11 commits into
mate-academy:masterfrom
dimon2202:develop
Open

add solution#991
dimon2202 wants to merge 11 commits into
mate-academy:masterfrom
dimon2202:develop

Conversation

@dimon2202
Copy link
Copy Markdown

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! 👏

here are some improvements:

  1. change title and favicon
Image
  1. items here should be without any discount
Image
  1. add some hover effects for these items
Image
  1. it's better to add cursor: pointer for all your interactive elements
Image
  1. back button should navigate user one step back, right now it is navigate to home page
Image
  1. numbers in both places should be the same
Image
  1. add some message after clicking on checkout button
Image
  1. this button doesn't work
Image
  1. it's better to add there some external links which will be opened in new tabs
Image

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