Skip to content

Conversation

@Cahllagerfeld
Copy link
Collaborator

@Cahllagerfeld Cahllagerfeld commented Apr 11, 2024

Description

✨ Add context to the sidebar to keep it open

Checklist

Please make sure that your pull request complies with the following requirements:

  • Code Quality:

    • 🧹 Code follows the project's coding guidelines.
  • Testing:

    • 🧪️ All existing tests pass successfully.
  • Documentation:

    • 📖 README and documentation are updated if necessary.
  • Review:

    • 👀 Code has been reviewed by a team member.
  • Security:

    • 🔒 Changes don't introduce security vulnerabilities.

Screenshots (optional)

grafik

@Cahllagerfeld Cahllagerfeld force-pushed the feature/update-sidebar branch from 42e1fe9 to c11436e Compare April 11, 2024 12:19
export function AppShell({ children }: { children: ReactNode }) {
return (
<div style={{ minHeight: "100vh", display: "flex", flexDirection: "column", width: "100%" }}>
<div style={{ minHeight: "1000px", display: "flex", flexDirection: "column", width: "100%" }}>
Copy link
Contributor

Choose a reason for hiding this comment

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

Just out of curiosity, this 1000px instead of a full vh is ok? 🤔

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Its just for the storybook preview, if I put it 100vh I need to scroll xD I just dont want to scroll when viewing it, so i set it to a fixed height 😅

@Cahllagerfeld Cahllagerfeld merged commit b0eed25 into main Apr 11, 2024
@Cahllagerfeld Cahllagerfeld deleted the feature/update-sidebar branch April 11, 2024 12:40
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.

3 participants