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

Bee crashes on Corridor selection #2006

Closed
marvin1099 opened this issue Dec 8, 2023 · 2 comments
Closed

Bee crashes on Corridor selection #2006

marvin1099 opened this issue Dec 8, 2023 · 2 comments
Labels
App Crash Something is causing the app to crash Bug Something isn't working correctly Done in next release This has been addressed in the next release

Comments

@marvin1099
Copy link
Contributor

marvin1099 commented Dec 8, 2023

BEE2 Application Version

2.4.43.0.dev+a2e7311d 64-bit

BEE2 Package Version

BEE2_v4.43.0

What operating system are you running on?

Linux

Logs and Reports

Traceback (most recent call last):
  File "ui_tk/corridor_selector.py", line 48, in <lambda>
  File "app/corridor_selector.py", line 275, in evt_selected
  File "ui_tk/corridor_selector.py", line 68, in set_highlight
AttributeError: module 'app.tk_tools' has no attribute 'LABEL_HIGHLIGHT_BG'

Additional information

This happens on my Linux box (Arch Linux | Garuda Linux), the app seams to crash if i select any Corridor (The style seems irrelevant).
The problem is not super urgent since the corridors are decorative but, I thought you might want to know about it.
Here a screenshot to what I'm referring to:
Screenshot_20231208_195210

@marvin1099 marvin1099 added App Crash Something is causing the app to crash Bug Something isn't working correctly labels Dec 8, 2023
@TeamSpen210 TeamSpen210 added the Done in next release This has been addressed in the next release label Dec 9, 2023
@TeamSpen210
Copy link
Member

This is because I didn't add a colour definition for Linux. I just copied the Window one, probably we'd actually need different ones for different themes/window managers/etc?

@marvin1099
Copy link
Contributor Author

Makes sense, but I'm not sure if you need different definitions for custom themes.
I thought the theme will make the decision as long as you tell the theme something like: select item.
Window Managers should also just do what do what the theme says so, I didn't think you need that.
But I'm not into this, so I could be wrong.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
App Crash Something is causing the app to crash Bug Something isn't working correctly Done in next release This has been addressed in the next release
Projects
None yet
Development

No branches or pull requests

2 participants