markteekman / accessible-astro-starter

An Accessible Starter Theme for Astro 4.0 including accessibility features such as landmarks, better focus-outline and skip-links navigation. Ships with Tailwind, Prettier and ESLint support.
https://accessible-astro.netlify.app/
MIT License
735 stars 136 forks source link

Page layout appears zoomed #60

Closed sangramrath closed 1 year ago

sangramrath commented 1 year ago

Hello Mark, I have an issue where the page layout appears zoomed (compared to the demo website). I have attached screenshots. The space around the page is less in the local run.

I am cloning the repo and run it without any modifications at the moment (I am exploring if it would fit my use case). I compared a few things and everything is stock, such as padding, grid layout and spacing etc.

It may be something very basic, hence I appreciate your time for this.

Demo Website Screenshot 1

Screenshot-of-demo-website-1


Local Run Screenshot 1

Screenshot-of-local-run-1


Demo Website Screenshot 2

Screenshot-of-demo-website-2


Local Run Screenshot 2

Screenshot-of-local-run-2
markteekman commented 1 year ago

Hi @sangramrath! Hmm, that's a very weird issue and hard for me to reproduce. When I run my local setup it looks exactly like the demo. What browser / OS are you using? And does this still apply when you run npm run build and upload the /dist to some server?

fayazmiraz commented 1 year ago

Hello Mark, I have an issue where the page layout appears zoomed

Just to be absolutely sure that there's no silly mistake here: Can you please press Ctrl+0 (on Windows/Linux) or CMD+0 (on mac) on both the local and the web version?

Browsers tend to remember the last zoom setting based on the URL. It's possible that one of those URL is zoomed out or zoomed in within your browser.

markteekman commented 1 year ago

@sangramrath I see there's indeed a difference, as mentioned in the linked issue above. I'll look into a solution when I can make some time.

saulopaiva commented 1 year ago

Hi!

It seams to be related with the order of tailwind base css is import.

Due the css specificity, it's better to import tailwind first and add the site css last.

This commit changes this behavior, and it's part of the Astro upgrade PR: #76.

markteekman commented 1 year ago

Solved in #76, thanks @saulopaiva!

sangramrath commented 11 months ago

Hello @markteekman thank you. I was travelling over the last few months and wasn't actively monitoring my notifications. Also thank you @saulopaiva for the commit and @fayazmiraz.