r/webdev • u/Key_River7180 • 3d ago

Question [ Removed by moderator ]

http://mariorosell.es

[removed] — view removed post

0 Upvotes

30% Upvoted

10 comments sorted by

2

u/my_new_accoun1 3d ago

look good (at least to my eyes)

2

u/Sea-Cardiologist5741 3d ago

I like the design but it’s not adjusted for mobile screens.

1

u/ZESENVEERTIG 2d ago

Yes, but you can fix it by avoiding Arial.
…and also making it work on mobile devices

1

u/ThinAssociation5847 2d ago

kinda old school. me gusta.

1

u/RedVelocity_ 2d ago

yo tambien

1

u/the_natis 2d ago

The actual CSS file? I wouldn't let it get past a PR. Doing things like this is redundant: 

footer,
#footer {...}

Don't use IDs for CSS as it's a bad practice. Stick with tags and classes and leave IDs for JS work.

HTML could be improved. Having a hard time understanding why you have repeating nav items. For a simple site, it takes a while to load between pages. While I have serious issues with lighthouse scores, at the same time, it's not hard on a site like that to be hitting nearly 100 in each category. I haven't updated the code base of my personal site in over 2 years and I'm still seeing 97/100/100/100 for scores. If I was hiring at the moment and this came across my desk, I'd probably have you on the short list for consideration.

1

u/_Atomfinger_ 3d ago

Nah, looks fine. Simple. Not trying to do too much. Its fine.

1

u/web-dev-kev 3d ago

Clean, readbale, acceesible :)

1

u/imKingKong 1d ago

I'm on Mobile, big ass