Skip to content

solution#534

Closed
LinaDo1609 wants to merge 0 commit intomate-academy:masterfrom
LinaDo1609:develop
Closed

solution#534
LinaDo1609 wants to merge 0 commit intomate-academy:masterfrom
LinaDo1609:develop

Conversation

@LinaDo1609
Copy link

Copy link

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

great job

  1. text style is different here
image
  1. disable text area resizing

  2. these links should open in a new tab

  3. add smooth scrolling to the page

Copy link

@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 job!

To improve:

  1. When I'm clicking on Send form' button, it gives me 405 error:
image

Hint: just re-use #15 from this instruction

  1. Would you mind to remove horizontal scroll?
image
  1. When I'm choosing one of sections from aside menu, nothing appears. Please fix it:
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.

3 participants