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

BCB: Add post comments styles #3553

Merged
merged 9 commits into from
Apr 2, 2021
Merged

BCB: Add post comments styles #3553

merged 9 commits into from
Apr 2, 2021

Conversation

jffng
Copy link
Contributor

@jffng jffng commented Mar 29, 2021

Starts to address #3546.

Which of these styles do you think should live with Gutenberg?

Styling in Gutenberg a bit tricky because the post comments block relies on the Core comments template: https://github.com/WordPress/gutenberg/blob/9b75a5c44e934f580282e2d16cc462ff4567cded/packages/block-library/src/post-comments/index.php#L33

Before After
Screen Shot 2021-03-29 at 1 00 16 PM Screen Shot 2021-03-29 at 12 56 25 PM

@scruffian scruffian requested a review from a team March 29, 2021 22:46
@danieldudzic
Copy link
Contributor

danieldudzic commented Mar 29, 2021

Here's the Gutenberg issue: WordPress/gutenberg#26864

I have added the button styling to the Reply link:
Comments_–_themes

@scruffian
Copy link
Member

Moved this to Gutenberg too: WordPress/gutenberg#26864

@jffng jffng changed the base branch from make/blank-canvas-blocks to trunk March 30, 2021 16:37
@jffng
Copy link
Contributor Author

jffng commented Mar 30, 2021

I removed some styles because of the PR to add some styles in Gutenberg.

@danieldudzic
Copy link
Contributor

@jffng Shouldn't this be merged with https://github.com/Automattic/themes/tree/make/blank-canvas-blocks rather than trunk?

@jffng
Copy link
Contributor Author

jffng commented Apr 1, 2021

I believe since we merged BCB to trunk, any changes to BCB can be made directly to trunk / synced to svn as soon as they're merged. make/blank-canvas-blocks is now a development branch for the new versions of Seedlet Blocks and Mayland Blocks.

@jffng jffng force-pushed the add/post-comments-styles branch from ad8a696 to bdbc67a Compare April 2, 2021 13:48
@jffng jffng merged commit ae57d02 into trunk Apr 2, 2021
@jffng jffng deleted the add/post-comments-styles branch April 2, 2021 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants