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

tabbed content broken with recent changes #1295

Closed
samdulam opened this issue Oct 27, 2022 · 6 comments
Closed

tabbed content broken with recent changes #1295

samdulam opened this issue Oct 27, 2022 · 6 comments

Comments

@samdulam
Copy link

samdulam commented Oct 27, 2022

With latest updates to tab.html and tabpane.html tabbed content looks like below
image

Manually creating those files in my project layouts/shortcodes using code from commit 1b10f6b fixes the issue.

@polarweasel
Copy link

@deining @chalin this is a big regression on tabpanes, and is breaking what I was hoping to fix by upgrading to 0.5.1...

@polarweasel
Copy link

Further details:

When you render a tabpane directly, it looks like what @samdulam posted, and the tabs are functional.

When you include a tabpane by reference using readfile, the appearance is the same, but the tabs are non-functional.

I'm playing with this in yugabyte/yugabyte-db#14694 if you want a live example. (Can't do the upgrade until this is resolved anyway, so we may as well use it for experimentation!)

@deining
Copy link
Collaborator

deining commented Oct 28, 2022

I just investigated your case, thanks for providing the repo!

With version 0.5.1, a breaking change in tabpanes was introduced: the parameter code was replaced by text.

@polarweasel: In your case this means, inside managed-quick-start-include.md you have to replace line 3:

tabpane code=false

with

tabpane text=true

Then have a look at your site preview, your tab should display correct again!

ToDo: This breaking change is undocumented yet, I will add a paragraph to CHANGELOG.md shortly.

Sorry for any confusion caused!

@deining
Copy link
Collaborator

deining commented Oct 29, 2022

ToDo: This breaking change is undocumented yet, I will add a paragraph to CHANGELOG.md shortly.

Done with PR #1298.

@polarweasel
Copy link

Thanks so much, @deining!

@chalin
Copy link
Collaborator

chalin commented Oct 31, 2022

Yes, thanks! Can this issue be closed?

@samdulam samdulam closed this as completed Nov 1, 2022
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

4 participants