Skip to content

Handle svg hero images in blocks/cover shortcode#1505

Merged
chalin merged 2 commits intogoogle:mainfrom
fekete-robert:block/cover-svg
Feb 2, 2024
Merged

Handle svg hero images in blocks/cover shortcode#1505
chalin merged 2 commits intogoogle:mainfrom
fekete-robert:block/cover-svg

Conversation

@fekete-robert
Copy link
Copy Markdown
Contributor

No description provided.

@chalin chalin force-pushed the block/cover-svg branch from 26c4aac to cda21e8 Compare May 31, 2023 15:49
@chalin chalin requested review from deining and geriom May 31, 2023 15:52
Copy link
Copy Markdown
Collaborator

@chalin chalin left a comment

Choose a reason for hiding this comment

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

Seems reasonable to me.
WDYT @deining @geriom?

@chalin chalin added this to the 23Q3 milestone May 31, 2023
@chalin chalin modified the milestones: 23Q3, 23Q4 Jul 27, 2023
@chalin chalin modified the milestones: 23Q4, 24Q1 Nov 3, 2023
@chalin chalin modified the milestones: 24Q1, 23Q4 Nov 30, 2023
@chalin chalin modified the milestones: 23Q4, 24Q1 Jan 11, 2024
Copy link
Copy Markdown
Collaborator

@chalin chalin left a comment

Choose a reason for hiding this comment

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

Whitespace clean up, otherwise LGTM

Comment thread layouts/shortcodes/blocks/cover.html Outdated
Co-authored-by: Patrice Chalin <chalin@users.noreply.github.com>
@fekete-robert
Copy link
Copy Markdown
Contributor Author

Thanks for the review, @chalin !

Copy link
Copy Markdown
Collaborator

@chalin chalin left a comment

Choose a reason for hiding this comment

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

Thanks for the PR @fekete-robert

@chalin chalin merged commit e22aec4 into google:main Feb 2, 2024
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.

2 participants