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

blocks/feature reworked to support use from .md files #727

Merged
merged 1 commit into from
Oct 19, 2021

Conversation

chalin
Copy link
Collaborator

@chalin chalin commented Oct 16, 2021

/cc @nate-double-u @celestehorgan

@google-cla google-cla bot added the cla: yes Indicates the PR's author has signed the CLA. label Oct 16, 2021
</div>
<h4 class="h3">
{{- .Get "title" | markdownify -}}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Reverting to the code used prior to #526 here.

@chalin chalin force-pushed the chalin-blocks-feature-2021-10-15 branch from 7108faf to e69f182 Compare October 16, 2021 00:41
@chalin chalin force-pushed the chalin-blocks-feature-2021-10-15 branch from e69f182 to efea297 Compare October 16, 2021 13:18
<h4 class="h3">
{{- .Get "title" | markdownify -}}
</h4>
<div class="mb-0">
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I kept the mb-0 class for now even though in some cases it isn't useful -- #724 (comment). I might have an alternative to suggest in a followup PR.

@LisaFC
Copy link
Collaborator

LisaFC commented Oct 19, 2021

This is great, thanks!

@LisaFC LisaFC merged commit 67e9c65 into google:master Oct 19, 2021
@chalin chalin deleted the chalin-blocks-feature-2021-10-15 branch October 19, 2021 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Indicates the PR's author has signed the CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feature shortcode no longer works inside of markdown files
2 participants