Dharmik_487• 1,740
@Dharmik48
Posted
Hey👋,
Your solution looks pretty good! I just have a couple of suggestions:
- Firstly, there is a vertical scroll bar in your solution and I don't think it should be there.
- And I suggest you to remove the default
outlineon focus state from buttons and links.
Apart from these you solution looks nice, keep it up👍
Marked as helpful
1
Sammy• 280
@soyed
Posted
@Dharmik48 Thanks for the feedback!
- do you mean the
overflow-y: autoon the page? if you mean that I think that is a default behaviour across web pages no? - I will remove the outline. Good catch 👍
0
