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

User preferences in Infotable #229

Open
georgeef opened this issue Nov 10, 2024 · 3 comments
Open

User preferences in Infotable #229

georgeef opened this issue Nov 10, 2024 · 3 comments
Labels
r: feature Request: New feature

Comments

@georgeef
Copy link
Contributor

georgeef commented Nov 10, 2024

Infotable contains the following fields:

  • DATEFORMAT: date format
  • CATEG_DELIMITER: category delimiter

These settings affect only the presentation of information in the user interface and not the content, i.e., they represent user preferences. They are better managed as application settings, and not as database settings.

The date format is traditionally managed by the locale settings at system level, since most likely the user wants the same setting across many applications. The category delimiter is specific to MMEX, but again, most likely the user wants the same setting across different database files.

Proposed behavior

Ignore DATEFORMAT and CATEG_DELIMITER in Infotable, i.e., do not read their value and do not offer an interface to update them.

Instead, manage the date format and category delimiter as application settings (@AppStorage).

Compatibility with MMEX Desktop/Android

There is no need for compatibility in application settings. The two fields above are just ignored and they are managed independently of the values in Infotable. Most likely the user would like to transfer them between different platforms as applications settings and not as database settings.

@georgeef
Copy link
Contributor Author

@guanlisheng
what do you think?

@georgeef georgeef added the r: feature Request: New feature label Nov 10, 2024
@guanlisheng
Copy link
Contributor

I am fine with leveraging it at the application level. it will also simplify Infotable as well.

@vomikan , @whalley , @renato-mmex , @wolfsolver for awareness.

@georgeef
Copy link
Contributor Author

CategoryDelimiter has been moved to app settings (Theme) with #241.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
r: feature Request: New feature
Projects
None yet
Development

No branches or pull requests

2 participants