8000 feat: Sidebar by nextchamp-saqib · Pull Request #341 · frappe/frappe-ui · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat: Sidebar #341

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

feat: Sidebar #341

wants to merge 10 commits into from

Conversation

nextchamp-saqib
Copy link
Member
@nextchamp-saqib nextchamp-saqib commented Jun 25, 2025 •
edited
Loading

CleanShot 2025-06-26 at 16 54 18@2x

CleanShot 2025-06-26 at 16 54 50@2x

@nextchamp-saqib
feat: initialize AppSidebar
e2a40b3
@nextchamp-saqib
refactor: AppSidebar
37fbed2 
• replace flat props into objects
• add types.ts & index.ts
• auto-collapse on mobile screens
• remove hardcoded dropdown options & suffix
• add story file
@nextchamp-saqib
feat: collapsible sidebar sections
4521b2d
@nextchamp-saqib
chore: rename AppSidebar folder to Sidebar
24d19ab
8000 @nextchamp-saqib
refactor: improve html structure & spacing
171c63f
@nextchamp-saqib
feat: add footer-items slot
508106a
@nextchamp-saqib nextchamp-saqib marked this pull request as ready for review June 26, 2025 11:43
@nextchamp-saqib nextchamp-saqib changed the title feat: AppSidebar feat: Sidebar Jun 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned 482A
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@nextchamp-saqib
0