Skip to content
This repository was archived by the owner on Nov 18, 2024. It is now read-only.

Conversation

@carolinan
Copy link
Contributor

@carolinan carolinan commented Sep 27, 2024

Description
I updated the order of the section styles from light to dark.
I placed the files in a new folder called sections.

I did not change the names, they are still called Style 1-5.

Screenshots
image

Before:
image

Testing Instructions

Add a group or columns block and confirm that the style sections have the correct color, but a different name and order than before the PR.
Add a few patterns that have a style section, and confirm that the color is the same as before the PR.

Contact location and link -yellow
Link in bio heading, paragraph, links and full-height image -purple
Hero podcast -pink
Logos -off white/ beige

@github-actions
Copy link

github-actions bot commented Sep 27, 2024

Preview changes

You can preview these changes by following the link below:

I will update this comment with the latest preview links as you push more changes to this PR.
⚠️ Note: The preview sites are created using WordPress Playground. You can add content, edit settings, and test the themes as you would on a real site, but please note that changes are not saved between sessions.

@carolinan carolinan requested a review from juanfra September 27, 2024 09:00
@carolinan carolinan marked this pull request as ready for review September 27, 2024 09:04
@github-actions
Copy link

github-actions bot commented Sep 27, 2024

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: carolinan <[email protected]>
Co-authored-by: juanfra <[email protected]>
Co-authored-by: richtabor <[email protected]>
Co-authored-by: jasmussen <[email protected]>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

Copy link
Member

@juanfra juanfra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good and working well to me. I left a small comment (nit) about ordering the sections by number in the different style variations, like they were before.

Screen.Recording.2024-09-27.at.11.51.38.mov

From my perspective, the order matches what @richtabor mentioned here (lightest to darkest), so that's good.

@juanfra juanfra linked an issue Sep 27, 2024 that may be closed by this pull request
@carolinan
Copy link
Contributor Author

carolinan commented Sep 28, 2024

Evening:
The colors are still not sorted from light to dark but dark to light:
image

Morning: is also kind of OK
image

It gets worse from there:

Noon:
image

Dusk:
image

Afternoon:
image

Twilight:
image

Sunrise:
image

Midnight:
image

@carolinan carolinan changed the title Try: Update the order of the style sections Try: Update the order of the section styles Sep 28, 2024
@carolinan carolinan merged commit 4496842 into trunk Sep 30, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add section styles

3 participants