-
Notifications
You must be signed in to change notification settings - Fork 11.6k
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
NestedFolders: Move New folder
into a drawer
#69706
Conversation
subtitle={parentFolderTitle ? `Location: ${parentFolderTitle}` : undefined} | ||
scrollableContent | ||
onClose={() => setShowNewFolderDrawer(false)} | ||
size="md" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just see my comment and decided whichever is best
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks Good 🚀
I wish CreateNewButton didn't have so much stuff in it, but then BrowseDashboardsPage would then just end up even bigger...
New folder
into a drawerNew folder
into a drawer
* make New folder a drawer * use sentence case * extract strings and update tests * use sm drawer
What is this feature?
New folder
functionality into a drawer (instead of a whole new page) to maintain contextNewFolderForm
, we can remove the old components once nested folders is liveNew Dashboard
/New Folder
to sentence case, e.g.New dashboard/
New folder`Why do we need this feature?
Who is this feature for?
Which issue(s) does this PR fix?:
Fixes #69693
Special notes for your reviewer:
Please check that: