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

No 'current' theme #805

Open
zwimer opened this issue Nov 30, 2022 · 5 comments · May be fixed by #806
Open

No 'current' theme #805

zwimer opened this issue Nov 30, 2022 · 5 comments · May be fixed by #806
Assignees
Labels

Comments

@zwimer
Copy link
Member

zwimer commented Nov 30, 2022

Description

When loading a theme in angr-management, there are 3 options: Light, Dark, and Current. Selecting Current can cause:

Traceback (most recent call last):
  File "/Users/zwimer/.virtualenvs/am-dev/lib/python3.10/site-packages/angrmanagement/ui/dialogs/preferences.py", line 151, in _on_load_scheme_clicked
    self._load_color_scheme(self._schemes_combo.currentText())
  File "/Users/zwimer/.virtualenvs/am-dev/lib/python3.10/site-packages/angrmanagement/ui/dialogs/preferences.py", line 146, in _load_color_scheme
    for prop, value in COLOR_SCHEMES[name].items():
KeyError: 'Current'

Steps to reproduce the bug

rm -rf ~/.config/angr-management
angr-management

Open preferences
"Themes and Colors"
Beside "Load Theme" select Current
Click "Load"

Environment

41b574e

Additional context

No response

@zwimer zwimer added the bug label Nov 30, 2022
@zwimer zwimer self-assigned this Nov 30, 2022
@zwimer
Copy link
Member Author

zwimer commented Nov 30, 2022

@mborgerson What is "Current" as a theme choice supposed to do? A no-op?

@zwimer
Copy link
Member Author

zwimer commented Nov 30, 2022

Is it supposed to be the System theme? If so, I think we should rename this to "System".

@mborgerson
Copy link
Member

'Current' is supposed to mean whatever is currently configured. e.g. if the user loads the Dark theme, Dark theme values are copied to Current

@zwimer zwimer linked a pull request Dec 1, 2022 that will close this issue
@zwimer
Copy link
Member Author

zwimer commented Dec 1, 2022

What is the rational for having "Current" when the loaded theme is already in the combobox selector and should be auto-selected when the preference tab opens? Or what does it add?

@zwimer
Copy link
Member Author

zwimer commented Dec 2, 2022

I'll remove the Current for now, this will is done in #806

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

Successfully merging a pull request may close this issue.

2 participants