Skip to content

Conversation

@ksuess
Copy link
Member

@ksuess ksuess commented May 30, 2025

Closes #6695 #6670


📚 Documentation preview 📚: https://volto--7157.org.readthedocs.build/


📚 Documentation preview 📚: https://plone-registry--7157.org.readthedocs.build/

@ksuess ksuess changed the base branch from main to seven May 30, 2025 08:22
@ksuess ksuess linked an issue May 30, 2025 that may be closed by this pull request
@ksuess ksuess marked this pull request as ready for review May 30, 2025 11:54
@ksuess ksuess requested review from pnicolli and sneridagh May 30, 2025 11:54
@ksuess
Copy link
Member Author

ksuess commented Jun 2, 2025

I moved the slot renderers for header and footer from 'App' to the 'publicui' layout. With this change the header and footer is on all publicui routes, not only on 'content'.

@sneridagh sneridagh added this to Seven Jun 23, 2025
<SlotRenderer name="header" content={content} location={location} />
</header>
<SlotRenderer name="main" content={content} location={location} />
<footer id="footer">
Copy link
Member

Choose a reason for hiding this comment

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

I'd bring back the footer in here, since it's valuable for the App component itself, in case somebody wants to use it anyways. It serve its purpose in Storybook story too. To make it more obvious: we won't be using it in Seven, since the Route outlet needs to be rendered in the middle (content).

* seven:
  [Seven] Block editor (#7228)
  Field and FieldWrapper component for widgets that have no field wrapper by default (#7213)
  Seven separate styles (#7225)
  Unify Container props (#7236)
  Recent PR 7221 docs fixes (#7226)
  Updated and unified react-aria version
  Addons registry style loader support for cmsui.css stylesheet in addons (#7221)
  Unified theming and styling for Storybook (#7220)
  Use American Engish spelling
  [components] File rearrangement (#7185)
  Unify implementations, remove custom BreadcrumbsPrimitive (#7196)
  seven menu dialog popover components (#7144)
  Small updates to package.json and LICENSE.md (#7165)
@sneridagh
Copy link
Member

@pnicolli @danalvrz @ksuess merging this one as well!! Please check if there's something wrong even after merging.

@sneridagh sneridagh merged commit 37ca979 into seven Jul 20, 2025
28 checks passed
@sneridagh sneridagh deleted the seven-sitemap branch July 20, 2025 10:23
@github-project-automation github-project-automation bot moved this to Done in Seven Jul 20, 2025
sneridagh added a commit that referenced this pull request Jul 26, 2025
* seven:
  Small fixes needed for CMSUI to work with the new arrangements
  [Seven] sitemap (#7157)
  [Seven] Block editor (#7228)
  Field and FieldWrapper component for widgets that have no field wrapper by default (#7213)
  Seven separate styles (#7225)
  Unify Container props (#7236)
  Recent PR 7221 docs fixes (#7226)
  Updated and unified react-aria version
  Addons registry style loader support for cmsui.css stylesheet in addons (#7221)
  Unified theming and styling for Storybook (#7220)
  Use American Engish spelling
  [components] File rearrangement (#7185)
  Unify implementations, remove custom BreadcrumbsPrimitive (#7196)
  seven menu dialog popover components (#7144)
  Small updates to package.json and LICENSE.md (#7165)
  Added cmsui @search loader (#7143)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[Seven Views] - Sitemap route

3 participants