amnestywebsite / humanity-theme

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

Epic - Swap old templates for FSE template #246

Closed AmnestyAM closed 3 weeks ago

AmnestyAM commented 5 months ago

Sprints allocated

Number of sprints: 1 (2+ for all of them but we will make a start) Link to sprint if known: TBC

Goals

Tickets

Think about (optional)

jaymcp commented 1 month ago

Known issues:

Gentyspun commented 1 month ago

Issues spotted on DEV, these may be unrelated to the epic tickets, and some may be fixable in the editor

jaymcp commented 1 month ago

Block spacing issues on most pages. (Must obvious on Homepage and /careers, example: http://bigbite.im/i/hR1TKR)

likely caused by this line: https://github.com/amnestywebsite/humanity-theme/blob/main/private/src/styles/pages/article/_base.scss#L92

might be worth scoping that selector to the single.html template

jaymcp commented 1 month ago

Inconsistent image and text sizing on /what-we-do as well as odd spacing (example: http://bigbite.im/i/eElb1j)

likely a result of moving to a pattern for LWI. can perhaps be addressed by either amending the pattern or modifying content. currently, the pattern is 3 columns with blocks within. perhaps we could do it with rows of three items instead. (e.g. title row, image row, etc)? (we may need to modify the migration tool if we do that)

jaymcp commented 1 month ago

/countries Explore By Region section displays too many regions/sub-regions/countries (example: http://bigbite.im/i/swj6Fs)

this should now be fixed. i'd forgotten to deploy the updated theme companion plugin. apologies.

jaymcp commented 1 month ago

Counter block displays total number below the counter on /join (example: http://bigbite.im/i/nI7ASB)

caused by a fixed line height here: https://github.com/amnestywebsite/humanity-theme/blob/develop/wp-content/themes/humanity-theme/theme.json#L379

we should either define line height as a unit-less value, or override it later.

Gentyspun commented 1 month ago

Issues found on Dev:

Admin/Dashboard

Patterns

Pages

BBTomA commented 1 month ago

Front-end issues found: Footer

Home page

What we do page

Get involved page

Donate page

Single news page

Single research page

jaymcp commented 1 month ago

The heading font size is a lot smaller than the live site http://bigbite.im/i/WzIyZw

It seems there's an issue with fluid typography; looking into it.

jaymcp commented 1 month ago

spotted an issue with remote attachments (e.g. when viewing ES document on EN site). i'm working on a solution, but it may take me a while to fix it completely.

jaymcp commented 1 month ago

spotted an issue with remote attachments (e.g. when viewing ES document on EN site). i'm working on a solution, but it may take me a while to fix it completely.

this is fixed

The heading font size is a lot smaller than the live site http://bigbite.im/i/WzIyZw

It seems there's an issue with fluid typography; looking into it.

this is also fixed

jaymcp commented 1 month ago

last outstanding issues that i'm aware of all relate to the new patterns intended to replace the custom card block. reviewing these now

Gentyspun commented 1 month ago

The below appears to be an issue on a number of staging sites

Issues found on https://isaidotorgstg.wpengine.com/en/

Issues found on https://exclimatestg.wpengine.com/

Issues found on https://iswpdocsstg.wpengine.com/

jaymcp commented 1 month ago

Whilst investigating custom card oddities, I noticed that we hadn't correctly migrated the colour variables in the branding plugin. I've got a PR in to address that. In the interim, some colours may look incorrect on DEV/STG.