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

Disable renaming settings files #95

Open
1 of 3 tasks
tpitkanen opened this issue Nov 22, 2020 · 0 comments
Open
1 of 3 tasks

Disable renaming settings files #95

tpitkanen opened this issue Nov 22, 2020 · 0 comments
Labels
enhancement New feature or request

Comments

@tpitkanen
Copy link
Member

tpitkanen commented Nov 22, 2020

Being able to rename settings files increases the complexity of code without providing much value for users. Issues caused by this feature:

  • settings files sometimes end up getting duplicated after naming (with both old and new names)
  • if there are multiple files of the same type, only the first file returned by the filesystem is used
  • finding settings files requires its own logic
  • all filetype information must be saved in the extension part because the filename can be whatever

Steps for phasing out settings renaming:

  • disable name field in settings dialogs
  • remove code for renaming settings files
  • change naming format (e.g. "Default.measurement" -> "measurement.json")
@tpitkanen tpitkanen added the enhancement New feature or request label Nov 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant