-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Implement folder setting to sort directories and files together and add option to toolbar sort flyout #9098
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
…d to toolbar sort flyout
Co-authored-by: Yair Aichenbaum <39923744+yaichenbaum@users.noreply.github.com>
I've restored the global setting and it seems to work, except that already open tabs are not updated when the setting is changed. |
Is this somethign you can copy from the option to show/hide hidden items? |
The problem is that for each folder visited if there is no custom setting, the value of the global setting is saved. So when the user changes the global setting again it will have no effect on that folder |
@manfromarce that's expected, to have the setting apply on all items you need to turn off the option to use individual preferences. |
Ok, I've done some tests and the combination of global and individual settings seems to work properly |
Awesome! Do you think the option to set the default value should be disabled when the option to use individual preferences is turned off? |
Alternatives:
|
It could be that when individual settings are disabled the new toolbar command is synced with the general setting (like show hidden/dot files) |
That can work, we can change it later if it gets confusing. |
@yaichenbaum @gave92 The settings should now be linked and refreshed correctly when individual settings are disabled, let me know what you think |
@manfromarce it's working great! We'll probably have to rearrange some options so that it's easier to see what's going on, but this is fantastic. |
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
OK, I think I can implement them. We could maybe add a new group in settings because "files and folders" would have too many entries |
We can add a group for sorting preferences |
Resolved / Related Issues
Items resolved / related issues by this PR.
Details of Changes
Validation
How did you test these changes?
Screenshots (optional)
