AlexisKenAlvarez / nextportfolio

My new portfolio, made with NextJS + TailwindCSS, 65% Done
https://alexisalvarez.info/
5 stars 0 forks source link

[bug] Cumulative Layout Shift #1

Open EXPITC opened 1 year ago

EXPITC commented 1 year ago

Hey👋! ,,

First cool project, here I just want to say or add some reference about (CLS), navigate between Home() or Projec() nav bar to Contact() make the layer shift to right which’s can be impact at SEO or just UX, this behavior tendly cause of scroll bar from previous render and take times to update the state like other (CLS) does, it would be great if the scroll bar can be hide or,,, show progress by fixed line which is not very visible next to it while the scroll bar in hidden state.

Additional note: In ur case you using parallax to showoff some cool project which is cool but mostly parallax case tend to hide the scroll bar or make it minimum less to see this is because the UI/UX want the end user to focus at the show and not having the distraction, having a scroll bar in scroll parallax effect make a small distraction or make end user eyes to not focus of what we want to show especially when the scroll bar to be more noticeable in this case, so this is why is many case they tend to hide or just show when need; last but not least having a fixed nav bar would be great to, cheers 🥳..

EXPITC commented 1 year ago

Overall this is just a feedback, it all back to you 🥂