-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
Feature: Added Copy functionality to sidebar and home page widgets #17969
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
base: main
Are you sure you want to change the base?
Feature: Added Copy functionality to sidebar and home page widgets #17969
Conversation
…nd home widget context menus.
|
This looks very good! I think we should disable it for the tags section and home item. |
…ction and the Home item in the sidebar.
@yaira2 That's good. Updated the code! |
|
I can confirm this is working for the sidebar and Quick Access widget. Two things I discovered:
|
| var itemContextMenuFlyout = new CommandBarFlyout() | ||
| { | ||
| Placement = FlyoutPlacementMode.Right | ||
| AlwaysExpanded = true |
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.
Is it possible to keep the FlyoutPlacementMode the same? This way it will be consistent across the app.
Resolved / Related Issues
Added Copy functionality to sidebar and home page widgets, allowing users to copy drives and folders to the clipboard for pasting elsewhere.
Closes #4262
Steps used to test these changes