Skip to content

task solution#732

Open
h-rostyslav wants to merge 3 commits into
mate-academy:masterfrom
h-rostyslav:develop
Open

task solution#732
h-rostyslav wants to merge 3 commits into
mate-academy:masterfrom
h-rostyslav:develop

Conversation

@h-rostyslav

Copy link
Copy Markdown

@etojeDenys etojeDenys left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

good job, to improve:

  1. those images should take up the full width of the screen
Image Image
  1. fix image placement (
Image
  1. remove the base outline
Image
  1. submit button should clear the input without refreshing the page
Image
  1. remove base style from links
Image
  1. when the menu is open, the bottom part should be covered
Image

example:

Image

@h-rostyslav
h-rostyslav requested a review from etojeDenys July 17, 2026 09:48

@Anton-Kuchmasov Anton-Kuchmasov left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hey, issues from prev review did not fixed:

Image

Did you forget to re-deploy your solution?

@h-rostyslav

Copy link
Copy Markdown
Author

Did you forget to re-deploy your solution?

yeah..my bad

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