Skip to content

Header slots - #7550

Open
arybakov05 wants to merge 26 commits into
sevenfrom
6673-seven-header-slots
Open

Header slots#7550
arybakov05 wants to merge 26 commits into
sevenfrom
6673-seven-header-slots

Conversation

@arybakov05

Copy link
Copy Markdown

@arybakov05

Copy link
Copy Markdown
Author

@sneridagh were the anontools, search widget and site actions slots supposed to be sub-slots of the header tools (with the same slot id) or is it fine to have them as individual slots? Also, I'm not sure if I should've used CSS modules or not, I just followed the way it was done in the Header component.

@arybakov05
arybakov05 marked this pull request as ready for review March 18, 2026 10:48
@arybakov05
arybakov05 requested a review from sneridagh March 18, 2026 10:55
@arybakov05
arybakov05 marked this pull request as draft March 18, 2026 13:33
@davisagli davisagli added this to Aurora Apr 7, 2026
@arybakov05 arybakov05 moved this from Backlog to In progress in Beethoven Sprint 2026 May 18, 2026
@arybakov05 arybakov05 self-assigned this May 18, 2026
@arybakov05
arybakov05 marked this pull request as ready for review May 18, 2026 14:06
@arybakov05 arybakov05 moved this from In progress to In review in Beethoven Sprint 2026 May 18, 2026
@arybakov05
arybakov05 requested a review from pnicolli May 18, 2026 14:07
@arybakov05 arybakov05 moved this to Ready for Review in Aurora May 18, 2026
@davisagli davisagli added this to the Seven milestone May 19, 2026
Comment thread packages/layout/slots/HeaderTools/AnonymousTools.tsx Outdated
Comment thread packages/layout/config/slots.ts
Comment thread packages/layout/slots/Header/Header.module.css
Comment thread packages/layout/slots/HeaderTools/SearchWidget.tsx
@sneridagh

Copy link
Copy Markdown
Member

@sneridagh were the anontools, search widget and site actions slots supposed to be sub-slots of the header tools (with the same slot id) or is it fine to have them as individual slots? Also, I'm not sure if I should've used CSS modules or not, I just followed the way it was done in the Header component.

The idea always was to model what we had back in the day with viewlets and viewlets managers, and the idea that you can nest them. The do not have to share the name or id, but they are meant to be semantic in the naming. So we have zones in the header (and in other slots) that we named back in the day, but we can name them now. Take a look at the classic viewlet managers to get the point:

https://classic.demo.plone.org/en/manage-viewlets

image

As long as they cover the "areas" left (logo), center (sections, menu) then right (actions, search widget, anontools language switcher?), is enough, because all are extensible and layered (stackable) by default. So we are good.

Comment thread packages/layout/config/slots.ts Outdated
});

config.registerSlotComponent({
name: 'Tools',

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Not sure about calling hem all Tools. The name of the slot component should be different... at least how I see it.

/>
<SlotRenderer
name="language-switcher"
name="headerTools"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm ok on doing this, and move it inside the tools.

@sneridagh

Copy link
Copy Markdown
Member

Important

Hello! Thanks for contributing to Plone Aurora! FYI, since June 5th, Plone Aurora lives in its own repository, thus, the seven branch does not accept any merges. If you want to merge your PR, please follow instructions: #8318
Sorry for the inconvenience!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Ready for Review
Status: In review

Development

Successfully merging this pull request may close these issues.

5 participants