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

Saving pages with image galleries in studio messes them up #1317

Closed
ChristopherChudzicki opened this issue May 11, 2022 · 2 comments
Closed
Assignees

Comments

@ChristopherChudzicki
Copy link
Contributor

ChristopherChudzicki commented May 11, 2022

Steps to Reproduce

  1. View page "Drawing up plans for the completed models" in studio RC:

    Note the closing {{</ image-gallery />}} at bottom is rendered just as {{}} in studio

  2. Edit the page title in studio and save it

Expected Behavior

Shortcodes should not change

Actual Behavior

Closing {{</ image-gallery />}} tag is converted to {{}}.

Related Issues

This is the "legacy shortcode" (shortocdes that only exist in legacy content) issue which Alice fixed in #1013. The code in that PR expected closing shortcodes to have the / right before the name. I.e.

// legacy shortcode fix expected
{{< /image-gallery >}}
// whereas we actually have
{{</ image-gallery >}}

Hugo seems to tolerate both, although the hugo documentation uses the first form.

Scope

Affects 322 pages across 75 courses. Editing those pages is probably pretty rare?

(All the other closing shortcodes we have, e.g., {{< /anchor >}} use the {{< /... form.)

@pdpinch
Copy link
Member

pdpinch commented May 19, 2022

Can this be closed?

@ChristopherChudzicki
Copy link
Contributor Author

ChristopherChudzicki commented May 19, 2022

@pdpinch I've been waiting till things get released to close stuff, but maybe that's just confusing. I just verified the commit on RC, so I'll close it.

Going out in #1365

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

No branches or pull requests

2 participants