222-responsive-icon-nav-css icon indicating copy to clipboard operation
222-responsive-icon-nav-css copied to clipboard

Published 20 hours ago • verified publisher icon fireship-io

Remove Overflow: scroll, Fix Transition var, Add .link-text transition

Open EmanueleSpadaro opened this issue 4 years ago • 0 comments

Hi! I tried to fix some project issues. Honestly I don't know why you merged to master a commit where overflow: scroll was applied to '.navbar', it literally broke the navbar layout as you can see in the gif. overflowfix

Secondly I fixed the transition time in .navbar since it was set as a constant of 600ms, I switched the value with the variable we previously declared at the start of the .css document.

Lastly, I made sure that the .link-text appears only when it doesn't exceed the navbar border while it's expanding on hover by using a combination of CSS properties like visibility, opacity and transition for both of them.

EmanueleSpadaro avatar Oct 07 '20 17:10 EmanueleSpadaro