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

Generalizing Rich Command Usage #11481

Closed
72 of 78 tasks
QuaintMako opened this issue Feb 26, 2023 · 23 comments · Fixed by #11821 or #11846
Closed
72 of 78 tasks

Generalizing Rich Command Usage #11481

QuaintMako opened this issue Feb 26, 2023 · 23 comments · Fixed by #11821 or #11846

Comments

@QuaintMako
Copy link
Contributor

QuaintMako commented Feb 26, 2023

Description

Thanks to @cinqmilleans' job, Files has now a cleaner way to handle commands across the solution. Rich commands now need to be implemented throughtout the whole solution.

This issue is used to keep track of which commands are made into Rich Commands.

Concerned code

The following commands must be transformed into Rich Commands (See BaseLayoutCommandsViewModel, ContextFlyoutItemHelper and ToolbarViewModel):

Gains

  • A more cohesive code.
  • The ability to call commands easily from anywhere in the code.

Requirements

  • Each of the commands previously cited have been transformed into Rich Commands.
  • The previous implementations of the commands are cleaned up.

Comments

No response

@yaira2
Copy link
Member

yaira2 commented Feb 26, 2023

Thank you for putting this together!

@yaira2 yaira2 moved this from 🆕 New to 🔖 Ready to build in Files task board Feb 26, 2023
@yaira2 yaira2 moved this from 🔖 Ready to build to 🏗 In progress in Files task board Feb 26, 2023
@hecksmosis
Copy link
Contributor

I'll do the sidebar pinned item ones

@hecksmosis
Copy link
Contributor

I'm working on some of these slowly

@QuaintMako
Copy link
Contributor Author

I'm tackling the 'Set As...' trio on my side.

@QuaintMako
Copy link
Contributor Author

Once we're done with implementing all the commands, we'll need to remember to do a round of cleaning (regrouping the commands together, make sure they are stored inside the appropriate folders, make sure there is no leftovers from the former implementations etc...)

@hecksmosis
Copy link
Contributor

Fyi, copy cut and delete heve already been implemented in #11504

@hishitetsu
Copy link
Member

Gridview Size Decrease and Gridview Size Increase have already been implemented in #11678.

@cinqmilleans
Copy link
Contributor

DuplicateTab/ Paste Items From Clipboard/(Pin/Unpin) Directory (To/From) (Favorites/Start) are implemented.
DragOver/Drop/Item Pointer Pressed must not be a command because they are only accessible by mouse (by nature).
A command to open the context menu would be handy.

@QuaintMako
Copy link
Contributor Author

Thanks for the update @cinqmilleans, I need to go through the code in more details to check out what needs to be made as commands. I'll change the checklist with your information!

@yaira2
Copy link
Member

yaira2 commented Mar 22, 2023

Search Unindexed Items

Isn't this a setting? (perhaps we can add support for settings but let's limit the scope to regular actions)

@hecksmosis
Copy link
Contributor

Shouldn't we make Close Tab a command?

@yaira2
Copy link
Member

yaira2 commented Mar 22, 2023

Shouldn't we make Close Tab a command?

Yes, with the ctrl + w shortcut.

@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in Files task board Mar 26, 2023
@yaira2 yaira2 reopened this Mar 26, 2023
@github-project-automation github-project-automation bot moved this from ✅ Done to 🆕 New in Files task board Mar 26, 2023
@yaira2 yaira2 moved this from 🆕 New to 🔖 Ready to build in Files task board Mar 26, 2023
@yaira2 yaira2 moved this from 🔖 Ready to build to 🏗 In progress in Files task board Mar 26, 2023
@yaira2

This comment was marked as resolved.

@QuaintMako

This comment was marked as resolved.

@0x5bfa

This comment was marked as resolved.

@hecksmosis
Copy link
Contributor

I'll look into open directory in new tab/Pane

@yaira2 yaira2 unpinned this issue May 7, 2023
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in Files task board May 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
7 participants