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

Add setting to settings to unrememeber default for drag and drop #2548

Open
EmmanuelMess opened this issue Jun 5, 2021 · 8 comments
Open
Labels
Area-UIUX Related to user interface/experience (e.g. dialogs, notifications). Issue-Easy (good first issue) Beginners welcome! Issues with relative low difficulty. Issue-Task (low) This isn't a bug, but should be dealt with.

Comments

@EmmanuelMess
Copy link
Member

EmmanuelMess commented Jun 5, 2021

Describe the bug
Drag and drop can be set as cut or copy from dialog, but it is not clear how to unselect the option later.

@EmmanuelMess EmmanuelMess added Issue-Task (low) This isn't a bug, but should be dealt with. Issue-Easy (good first issue) Beginners welcome! Issues with relative low difficulty. Area-UIUX Related to user interface/experience (e.g. dialogs, notifications). labels Jun 5, 2021
@VishalNehra
Copy link
Member

@EmmanuelMess That's kinda unnecessary. It'll make settings more confusing.
What's wrong in setting it at time of drag / drop?

@EmmanuelMess
Copy link
Member Author

EmmanuelMess commented Jun 5, 2021

What's wrong in setting it at time of drag / drop?

Setting it is not the issue, un setting is. Maybe the description should be better. Edit: fixed description.

@VishalNehra
Copy link
Member

There is already an option for this in settings, check clear cache. That should clear this one as well.

@EmmanuelMess
Copy link
Member Author

There is already an option for this in settings, check clear cache. That should clear this one as well.

This is not cache. That shouldn't be the way to clear a setting.

@EmmanuelMess EmmanuelMess reopened this Jul 11, 2021
@VishalNehra
Copy link
Member

Can rename 'Clear cache' to 'clear defaults' then. The functionality remains the same. It clears any defaults set, be it in drag/drop or the open apps bottom sheet. I think a few more defaults are included.

@EmmanuelMess
Copy link
Member Author

@VishalNehra that doesn't give the user the option to just unset this setting, how much is reset by clicking that? Why was that button implemented?

@VishalNehra
Copy link
Member

Giving option to reset just this setting isn't practical imo. That's like giving too much options, it'll lead to confusing interface.
Most apps provide such option named 'clear cache / defaults' that generally clears all the settings and defaults set within that app. This option follows that principal.

@EmmanuelMess
Copy link
Member Author

This can be fixed with with windows like Telegram or Anki have IMO.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-UIUX Related to user interface/experience (e.g. dialogs, notifications). Issue-Easy (good first issue) Beginners welcome! Issues with relative low difficulty. Issue-Task (low) This isn't a bug, but should be dealt with.
Projects
None yet
Development

No branches or pull requests

2 participants