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

Use JSON for configuring the add-on settings #2

Open
Archomeda opened this issue Feb 2, 2019 · 0 comments
Open

Use JSON for configuring the add-on settings #2

Archomeda opened this issue Feb 2, 2019 · 0 comments
Labels
Area: Add-on API Related to the native add-on C API Duration: Long Between 8 and 24 hours of estimated work Priority: High High priority Status: In progress This is being worked on Type: Feature An addition to the project
Milestone

Comments

@Archomeda
Copy link
Owner

At first I thought that the current API would be good for configuring the optional settings window for add-ons. However, I feel that it's limited and not intuitive to work with. So I'm going to experiment to see if providing a JSON string is better instead.

This will most likely also change how the settings are stored for each add-on. Right now it's up to the add-on to store its settings (it doesn't matter if the loader is providing the GUI, or the add-on itself). But that doesn't feel intuitive either.

So the idea is to have 3 implementation options available to each add-on:

  • GUI provided by the loader, settings saved/loaded by the loader (settings definitions provided through JSON by the add-on, add-on gets callbacks when settings changes)
  • GUI provided by the add-on, settings saved/loaded by the add-on (add-on gets callback whenever the user wants to open the settings panel)
  • No add-on settings

Note that only 1 of these options can be used at one time for each add-on individually, but each add-on can choose whatever they want.

@Archomeda Archomeda added Type: Feature An addition to the project Status: In progress This is being worked on Area: Add-on API Related to the native add-on C API Priority: High High priority Duration: Long Between 8 and 24 hours of estimated work labels Feb 2, 2019
@Archomeda Archomeda added this to the 1.0.0-beta.1 milestone Feb 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Add-on API Related to the native add-on C API Duration: Long Between 8 and 24 hours of estimated work Priority: High High priority Status: In progress This is being worked on Type: Feature An addition to the project
Projects
None yet
Development

No branches or pull requests

1 participant