Skip to content
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

Add & sync download subpages #131

Merged
merged 9 commits into from Oct 17, 2022
Merged

Add & sync download subpages #131

merged 9 commits into from Oct 17, 2022

Conversation

ryelle
Copy link
Contributor

@ryelle ryelle commented Oct 5, 2022

Fixes #129 — This branch has all the download subpages in one place. This contains #132 & #134, and uses WordPress/wporg-parent-2021#48.

This enables the syncing for all 4 download subpages: Releases, Nightly, Counter, Source. Once we merge this, the new design can be enabled on these pages.

There was discussion about the subnavigation bar on #129 — I've added a basic iteration but the actual alignment can be iterated on in the editor after merge, if necessary.

Screenshots

Beta/Nightly Counter Releases Source
beta-desktop counter-desktop releases-desktop source-desktop
beta-mobile counter-mobile releases-mobile source-mobile

Note, the above screenshots were taken before the mobile nav button was updated to use the down-chevron - it looks like this now:
Screen Shot 2022-10-14 at 4 55 38 PM

Mobile "dropdown" style nav open
source-mobile-nav-open counter-mobile-nav-open

How to test the changes in this Pull Request:

Visit the 4 download subpages on your local site.

* Use the new theme for all `wp*` endpoints

This enables the new theme on `/wporg/v1/`  endpoints as well as core `/wp/v2/`.

* Add Download Counter block & endpoint

* Style the download counter block

* Add a shortcode for the latest branch constant.

* Update Counter page pattern

* Catch errors in JS

* Rename shortcode

Technically the "latest" branch would be the one in development, so rename this to better reflect its value.

* Use charcoal-2 background color

* Use `textContent` for safety, since the count value is plain text

* Update font to new monospace font

* Use the extra-light font weight
* Initial Release Tables block

* Add the toggle view for older releases

* Visually hide table headers, add responsive styles

* Always make sure the headers are hidden

* Align all the links, even when they wrap to new lines

* Add a function check

* Remove unnecessary settings

* Replace bar with middot in hash links
@ryelle ryelle marked this pull request as ready for review October 14, 2022 20:58
@ryelle
Copy link
Contributor Author

ryelle commented Oct 14, 2022

All the code parts have been reviewed in other PRs, so the only this needs is a design review. Feel free to tag in any other designers who should be involved.

Hopefully if there's anything minor (or anything in the content), we can tweak that in the editor after merge, and re-sync the content.

@jasmussen
Copy link

At a glance, looks good to me! The download counter on mobile may look a bit tall, in how it appears to push some of the footer links to be in an awkward place mid-fold, but perhaps that's related to the adminbar? Probably fine.

@javierarce
Copy link
Collaborator

javierarce commented Oct 17, 2022

A minor thing that could be tweaked after the merge: if possible, I would just reduce the font size of the "Number of WordPress 6.0 downloads" and increase the margin between that line and the counter to get closer to the original design.

The rest looks good to me ✨

@ryelle
Copy link
Contributor Author

ryelle commented Oct 17, 2022

@jasmussen @javierarce I think you can tweak those things once I've deployed this, since the issues you've raised can be fixed in the editor. Once you've got it the way you want, I can re-sync the patterns.

@ryelle ryelle merged commit 58cfe45 into trunk Oct 17, 2022
@ryelle ryelle deleted the add/download-subpages branch October 17, 2022 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Component] Theme Templates, patterns, CSS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Download Subpages
3 participants