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

[4.x] Wire up global site picker groups #9565

Closed
wants to merge 2 commits into from
Closed

[4.x] Wire up global site picker groups #9565

wants to merge 2 commits into from

Conversation

jackmcdade
Copy link
Member

@jackmcdade jackmcdade commented Feb 21, 2024

I have a proof of concept working for the global site picker, but can't work out the best way to not duplicate a bunch of this logic in JS everywhere we need to use it. Could use an assist here.

  • Global Site Picker
  • Entries Sidebar
  • Terms Sidebar

@jackmcdade jackmcdade marked this pull request as draft February 21, 2024 20:37
So it can be re-used throughout the JS.
@jackmcdade
Copy link
Member Author

Okay dang, the way we fetch and work with the sites list in the Entries/Terms sidebar is so wildly different than that Global Header that even with a unified config I can't untangle it. Closing this for now as it feels like too big of a rabbit hole and probably needs a wider view of the architecture. Stinks cuz I just wanna plop some HTML somewhere but it just ain't that simple.

@jackmcdade jackmcdade closed this Feb 29, 2024
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