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

Inconsistent shortcode rendering #1947

Closed
bemyak opened this issue Jul 31, 2022 · 2 comments
Closed

Inconsistent shortcode rendering #1947

bemyak opened this issue Jul 31, 2022 · 2 comments
Labels
bug done in pr Already done in a PR

Comments

@bemyak
Copy link
Contributor

bemyak commented Jul 31, 2022

Bug Report

Environment

Zola version: 0.16.0, next
OS: Linux

Expected Behavior

Shortcodes should render the same way each invocation

Current Behavior

The youtube shortcode in the attached website produces different results from build to build. Sometimes it renders correctly into:

<figure class="image is-16by9">
    <iframe class="has-ratio" frameborder="0" allowfullscreen width="100%"
        src="https://www.youtube-nocookie.com/embed/dQw4w9WgXcQ"></iframe>
</figure>

<div class="comment">
    <em><p>test</p>
</em>
</div>

Sometimes it's just

<div >
    <iframe src="https://www.youtube-nocookie.com/embed/dQw4w9WgXcQ" webkitallowfullscreen mozallowfullscreen allowfullscreen></iframe>
</div>

Step to reproduce

Unzip the archive, and run zola build && cat public/post/index.html around 5-10 times. Observe that the output is slightly different.

@bemyak
Copy link
Contributor Author

bemyak commented Aug 4, 2022

@Keats, any tips here? I'm not sure how to debug this, it looks like a concurrency issue 🤷

@Keats
Copy link
Collaborator

Keats commented Aug 4, 2022

Looks like it's somehow conflicting with the built-in youtube shortcode and rendering that one instead, which is weird as it should have been replaced by the custom one.

@Keats Keats added the bug label Aug 4, 2022
Keats added a commit that referenced this issue Aug 7, 2022
@Keats Keats added the done in pr Already done in a PR label Aug 7, 2022
@Keats Keats closed this as completed in abafe66 Aug 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug done in pr Already done in a PR
Projects
None yet
Development

No branches or pull requests

2 participants