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

OEL-1618: Update BCL to 0.23.0 #242

Merged
merged 12 commits into from
Jun 23, 2022
Merged

OEL-1618: Update BCL to 0.23.0 #242

merged 12 commits into from
Jun 23, 2022

Conversation

escuriola
Copy link
Contributor

donquixote
donquixote previously approved these changes Jun 1, 2022
Copy link
Contributor

@donquixote donquixote left a comment

Choose a reason for hiding this comment

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

This one seems ok. The other prs still need work.

@donquixote
Copy link
Contributor

donquixote commented Jun 13, 2022

I think we need a follow-up for featured media paragraph + pattern.
The pattern template is weird because it contains so much custom html instead of using bcl.

The heading in the pattern is h3. To align with other paragraphs it should be h2.
In netlify it is h2.fw-bold, which is probably a mistake: https://oelibrary.netlify.app/default/?path=/story/paragraphs-featured-media--featured-item

EDIT: Actually it is ok on the "Landing page" where the paragraph is used: https://oelibrary.netlify.app/default/?path=/story/features-landing-page--default

Let's fix here, no need for separate issue.

donquixote
donquixote previously approved these changes Jun 15, 2022
brummbar
brummbar previously approved these changes Jun 22, 2022
Copy link
Contributor

@drishu drishu left a comment

Choose a reason for hiding this comment

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

on behalf of @donquixote

@escuriola escuriola merged commit 6118b5e into 1.x Jun 23, 2022
@escuriola escuriola deleted the OEL-1618 branch June 23, 2022 12:34
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.

None yet

4 participants