search

amnestywebsite / humanity-theme

The Amnesty International WordPress Humanity Theme
https://wordpresstheme.amnesty.org/
Other
38 stars 10 forks source link

Bug - Post Template alignment - Main content not aligned with Featured Image/Hero/Main Menu on desktop #458

Open carolinejcourtney opened 4 weeks ago

carolinejcourtney commented 4 weeks ago

Describe the bug
Not sure if this is a bug or a change, but logging it anyways...

It looks like there is a gutter on the sides of the main content container in the Post template. This means that the main content doesn't line up with the Featured Image, Hero and Main Menu.

https://www.amnesty.org/en/latest/news/2024/10/myanmar-bangladesh-rohingya-community-facing-gravest-threats-since-2017/

This might be a result of the move to FSE, or it was pre-existing. But I imagine we want this to be aligned on larger screens (where there is a bit of white space on the sides of the Hero, Featured Image and Main Menu). On mobile or smaller screens, the main content container should be indented.

Feel free to delete this issue if this is a new change rather than a bug, I just happened to notice it when I was testing out .org this week.

jaymcp commented 4 weeks ago

Hey @carolinejcourtney this was a requested change a while back; originally in https://github.com/amnestywebsite/amnesty-wp-theme/issues/2595, and then tracked in https://github.com/amnestywebsite/amnesty-wp-theme/issues/2916. I think it was

carolinejcourtney commented 3 weeks ago

Ah ok - Then feel free to ignore this one then! Might keep it logged and speak to @AmnestyAM about it another time to see if we want to adjust it slightly so that the extra indent is only there on mobile.