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

Further settings #282

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Further settings #282

wants to merge 2 commits into from

Conversation

donni106
Copy link
Member

This was prepared once and is on hold since then.

SVA-91

- added a new setting for switching a theme with more contrast
  - setting can be enabled/disabled through global settings
- currently nothing happens on switching, just the state
  is saved
  - the contrast view needs to be discussed

SVA-91
- added a new setting for selecting data providers
  - setting can be enabled/disabled through global settings
- `SettingsDataProvider` is based on `SettingsListItem` and
  opens a dialog, which shows all data provider with checkboxes
  - created `Checkbox` based on `Radiobutton`
- currently nothing happens on selecting, just the state
  is saved
  - the behavior after selection needs to be discussed

SVA-91
@donni106 donni106 added enhancement New feature or request question Further information is requested labels Jul 23, 2021
@donni106 donni106 self-assigned this Jul 23, 2021
@codeclimate
Copy link

codeclimate bot commented Jul 23, 2021

Code Climate has analyzed commit 1cef1a4 and detected 15 issues on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 2
Duplication 9
Bug Risk 4

View more on Code Climate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant