-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Feature: Added file tags widget #10477
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
Conversation
Are any return welcomed / needed so far? |
We can use some feedback on the issue requirements |
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.
LGTM
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 src/Files.App/Assets/FilesOpenDialog/FilesLauncher.exe
supposed to be pushed?
Spacing seems different in NativeFolderWatcher.cs
, NativeStorage.cs
, NativeFile.cs
, FtpStorageService.cs
, FtpStorageFile.cs
, FtpStorable.cs
src/Files.Backend/ViewModels/Widgets/FileTagsWidget/FileTagsContainerViewModel.cs
Show resolved
Hide resolved
src/Files.Backend/ViewModels/Widgets/FileTagsWidget/FileTagsWidgetViewModel.cs
Show resolved
Hide resolved
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 overall and promising, I need to go play with it a bit in the actual application to see if there is no issue. Otherwise good work!
One last question, does the FileLauncher.exe
really needs to be updated?
src/Files.App/Filesystem/StorageEnumerators/UniversalStorageEnumerator.cs
Show resolved
Hide resolved
@d2dyno1 can you fix the merge conflicts? |
It was built automatically by VS. Perhaps .gitignore should be updated to exclude it? @QuaintMako |
Don't worry about the launcher, changes to it shouldn't really matter. |
Resolved / Related Issues
Changes
This PR replaces Bundles widget with new tags widget based on File Tags. This PR is a work in progress.
About the widget
The widget is fully functional, but the plan is to add a toggle switch only after there is support for editing tags within the app. In the meantime, you can manually enable the widget by editing the settings file (settings > advanced > edit settings file). Please note that the app should be closed before making changes to the file.
Validation
How did you test these changes?