search

WordPress / community-themes

A collection of Block Themes built by the WordPress community.
80 stars 33 forks source link

TT1 blocks: cleanup CSS #28

Closed MaggieCabrera closed 1 year ago

MaggieCabrera commented 1 year ago

Closes #24

This PR is a WIP. We are aiming to remove most of (if not all) the CSS from the theme.

carolinan commented 1 year ago

Should we merge this and the theme.json update to one branch (I don't know how but I think it would be easier)

MaggieCabrera commented 1 year ago

Should we merge this and the theme.json update to one branch (I don't know how but I think it would be easier)

I don't, either. We could make one PR based on the other but I don't know how much we need to do that, this PR so far is not touching on theme.json, I was expecting the theme.json PR to be merged before this one anyway

carolinan commented 1 year ago

I'm finding it difficult to see how the theme.json changes affect the theme because that PR doesn't remove the CSS.

MaggieCabrera commented 1 year ago

I'm finding it difficult to see how the theme.json changes affect the theme because that PR doesn't remove the CSS.

I'm not working on this today, if you want you can cherry pick my commits to your PR and we can close this one

MaggieCabrera commented 1 year ago

Merged these changes inside https://github.com/WordPress/community-themes/pull/29 so we just work on that one branch. Closing this.