Skip to content

Conversation

@Polo2
Copy link
Member

@Polo2 Polo2 commented Sep 29, 2025

WIP: Require #405 to be merged first


Preliminary work: => moved in dedicated PR #405

  • Display slot 'before' if provides, with <%= slotted :before %>
  • Favor tag h2 for this slot, to have only one h1 per page

Version selector

Be generic, provide a list of selectable version in data.sidebar_versions When sidebar_versions are provided, display a (raw) select HTML.

TODO:

  • improve CSS for this select (dedicated component)
  • improve behavior: add link between v3.1 & v3.2 support smart SEO logic when page is missing

**Preliminary work:**

- Display slot 'before' if provides, with `<%= slotted :before %>`
- Favor tag h2 for this slot, to have only one h1 per page

**Version selector**

Be generic, provide a list of selectable version in `data.sidebar_versions`
When sidebar_versions are provided, display a (raw) select HTML.

TODO:
- improve CSS for this select (dedicated component)
- improve behavior:
  add link between v3.1 & v3.2
  support smart SEO logic when page is missing
@netlify
Copy link

netlify bot commented Sep 29, 2025

Deploy Preview for bump-content-hub ready!

Name Link
🔨 Latest commit 70e7479
🔍 Latest deploy log https://app.netlify.com/projects/bump-content-hub/deploys/68da9031b39d4e0008a963ee
😎 Deploy Preview https://deploy-preview-404--bump-content-hub.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Polo2 Polo2 self-assigned this Sep 29, 2025
<%= resource.data.sidebar_title %>
<span class="version"><%= resource.data.sidebar_version %></span>
</h1>
<%# TODO: move in dedicated component %>
Copy link
Member Author

Choose a reason for hiding this comment

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

This can be done in dedicated commit, and then fix the missing slot in this PR only

@Polo2
Copy link
Member Author

Polo2 commented Sep 30, 2025

This PR can not be found anymore, content moved to #406

@Polo2 Polo2 closed this Sep 30, 2025
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.

2 participants