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

fix(docsite-v9): move theme picker #25385

Conversation

tudorpopams
Copy link
Contributor

Current Behavior

The theme picker is currently positioned on the top bar of the docsite making it unusable for embedded pages.

image

New Behavior

By moving the theme picker under each component's title bar, we will have access to it both on iframes and normal docsite.

image

@tudorpopams tudorpopams requested a review from a team as a code owner October 26, 2022 15:10
@tudorpopams tudorpopams self-assigned this Oct 26, 2022
@tudorpopams tudorpopams requested review from a team as code owners October 26, 2022 15:10
@fabricteam
Copy link
Collaborator

fabricteam commented Oct 26, 2022

📊 Bundle size report

🤖 This report was generated against f6f3e4b27f850577890dce15000bf8d4dedc4381

@size-auditor
Copy link

size-auditor bot commented Oct 26, 2022

Asset size changes

Size Auditor did not detect a change in bundle size for any component!

Baseline commit: f6f3e4b27f850577890dce15000bf8d4dedc4381 (build)

@codesandbox-ci
Copy link

codesandbox-ci bot commented Oct 26, 2022

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit c2356ab:

Sandbox Source
@fluentui/react 8 starter Configuration
@fluentui/react-components 9 starter Configuration

@tudorpopams tudorpopams force-pushed the fix/storybook-theme-changer-position branch from c8b9e9d to c2356ab Compare October 26, 2022 17:06
@tudorpopams tudorpopams requested a review from Hotell November 1, 2022 13:15
@tudorpopams tudorpopams merged commit cd78876 into microsoft:master Nov 2, 2022
@micahgodbolt
Copy link
Member

I don't think it's worth blocking right now, but maybe add a backlog item to come back and improve the layout a bit? It'd be nice to not have everything so stacked.

Moving the Button out to the right will take a bit of work (especially to make sure it works over viewports and title lengths).

image

Or maybe make it part of the sidebar (though again need to solve for smaller screen where the sidebar isn't present)

image

marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Nov 2, 2022
* master: (23 commits)
  fix(docsite-v9): move theme picker under component title so it can be visible on embedded pages (microsoft#25385)
  applying package updates
  feat: Implement child render function for DataGrid rows (microsoft#25476)
  fix(useTable): sort should adapt to enhanced row types (microsoft#25487)
  applying package updates
  feat: positioning should happen out of React lifecycle (microsoft#25456)
  applying package updates
  chore(react-link): migrate to new package structure (microsoft#25471)
  chore(react-input): migrate to new package structure (microsoft#25469)
  fix glob pattern for syncpack (microsoft#25465)
  chore(react-label): migrate to new package structure (microsoft#25470)
  chore: bump Griffel to latest (microsoft#25412)
  chore: add few small toolbar improvements (microsoft#25468)
  docs: fix small typos (microsoft#25464)
  chore: fix dependencies in @fluentui/react-toolbar (microsoft#25466)
  feat: v0 menu style migration from v9 (microsoft#25012)
  applying package updates
  Update List to render children on first render() call (microsoft#25331)
  Scaffold react-skeleton package (microsoft#25435)
  fix(public-docsite): Changing crossplatform urls to use 'cross' instead of 'crossplatform' (microsoft#25437)
  ...
NotWoods pushed a commit to NotWoods/fluentui that referenced this pull request Nov 18, 2022
Hotell pushed a commit to Hotell/fluentui that referenced this pull request Feb 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants