Skip to content

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

Merged
merged 40 commits into from
Jan 31, 2023
Merged

Feature: Added file tags widget #10477

merged 40 commits into from
Jan 31, 2023

Conversation

d2dyno1
Copy link
Member

@d2dyno1 d2dyno1 commented Nov 15, 2022

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?

  • Built and ran the app
  • Tested the changes for accessibility

@d2dyno1 d2dyno1 changed the title Initial changes Replaced Bundles with File Tags widget Nov 15, 2022
@QuaintMako
Copy link
Contributor

Are any return welcomed / needed so far?

@yaira2
Copy link
Member

yaira2 commented Dec 18, 2022

Are any return welcomed / needed so far?

We can use some feedback on the issue requirements

@cinqmilleans cinqmilleans mentioned this pull request Dec 27, 2022
2 tasks
@d2dyno1 d2dyno1 requested a review from yaira2 January 28, 2023 21:10
yaira2
yaira2 previously approved these changes Jan 29, 2023
Copy link
Member

@yaira2 yaira2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@ferrariofilippo ferrariofilippo left a 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

Copy link
Contributor

@QuaintMako QuaintMako left a 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?

@yaira2
Copy link
Member

yaira2 commented Jan 30, 2023

@d2dyno1 can you fix the merge conflicts?

yaira2
yaira2 previously approved these changes Jan 30, 2023
@d2dyno1
Copy link
Member Author

d2dyno1 commented Jan 30, 2023

One last question, does the FileLauncher.exe really needs to be updated?

It was built automatically by VS. Perhaps .gitignore should be updated to exclude it? @QuaintMako

@yaira2
Copy link
Member

yaira2 commented Jan 30, 2023

Don't worry about the launcher, changes to it shouldn't really matter.

@yaira2 yaira2 added ready to merge Pull requests that are approved and ready to merge and removed needs - code review labels Jan 31, 2023
@yaira2 yaira2 merged commit da317c8 into main Jan 31, 2023
@yaira2 yaira2 deleted the f_bndlsv2 branch January 31, 2023 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Pull requests that are approved and ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature: Replace bundles with tags widget
4 participants