-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Initial pass for About page #3433
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Asked about the design items in slack: https://wordpress.slack.com/archives/C02S78ZAL/p1665506685176209 |
|
There are still remaining design issues but we can address those post RC — it's nothing that would change the strings. |
|
committed in r54499 |
|
Per some recent milestone discovery, there's a feature highlighted on the About Page that will not be part of 6.1. Therefore, we will need to remove the entire bullet from the About Page copy.
|
|
Additionally, we need to remove the following sentence from the copy, because the number value isn't an accurate representation of the material changes (fixes and enhancements).
I will sponsor a discussion separately from this ticket as to how best count and communicate product update (bug fixes and enhancement) counts in communications for future assets. |


This PR moves the copy from the doc into code, using the design from Figma.
The header is still the 6.0 version, last I heard the design here is still in progress.Header is still in progress, but I've updated the PR to remove the "6.0" image.All the content and links should be in place, but none of the images have been exported yet (so they're red circles). The final images aren't necessary for RC.
I changed the layout of the last section in the icon grid — "WordPress 6.1 performance highlights" was very long, so I combined 3 smaller sections to line them up better (bonus, those were missing icons, now they don't need any).
One last section, "Block template parts in classic themes", had a repeat icon, so that's also been placeholder-ified with a red circle.
This is missing translator comments for the strings.
Trac ticket: https://core.trac.wordpress.org/ticket/56357
This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the Core Handbook for more details.