Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

Remove default color palette #105

Merged
merged 1 commit into from
Aug 28, 2023
Merged

Remove default color palette #105

merged 1 commit into from
Aug 28, 2023

Conversation

richtabor
Copy link
Member

@richtabor richtabor commented Aug 28, 2023

Description

Proposing that we remove the default color palette, in support of making the Twenty Twenty Four experience much more tailored. It's less distracting without them, and we don't use those values anyhow.

We'll continue to refine the theme colors, but in the meantime the defaults seem out of place.

Screenshots

Before After
CleanShot 2023-08-28 at 09 50 13 CleanShot 2023-08-28 at 09 49 44

Testing

  1. Activate the theme.
  2. Add a page with a paragraph.
  3. Select "Color" to see the default colors not displaying.

@melchoyce
Copy link
Contributor

Agreed 👍

@jeffikus
Copy link
Collaborator

+1 from me, any thoughts @MaggieCabrera @luminuu ?

Copy link
Collaborator

@jeffikus jeffikus left a comment

Choose a reason for hiding this comment

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

I'm fine with this, but I think it would be good to get feedback from the theme leads before we merge this. Approving it for now though.

Copy link
Collaborator

@MaggieCabrera MaggieCabrera left a comment

Choose a reason for hiding this comment

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

I like this idea

@MaggieCabrera MaggieCabrera merged commit 33567ae into trunk Aug 28, 2023
2 checks passed
@juanfra juanfra deleted the try/no-default-palette branch August 30, 2023 15:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants