-
Notifications
You must be signed in to change notification settings - Fork 74
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
Fails to store settings in Manjaro Linux #17
Comments
Ah, found the culprit. 💥 🎉 if (name.contains("&")) {
name.replace("&", "");
} I don't want to mess with this issue anymore and use this workaround 😠 |
Ah, the investigation hasn't been stopped folks! As I have committed the work workaround, there shouldn't be any problems for now but I will try to get myself an another solution. |
I think I have figured out the main thing: So this is not a KDE environment based issue. Most distributions use Qt 5.9 like Ubuntu 17.10 and so we didn't hit this issue. But Arch distributions use latest Qt and we hit this issue really hard 💥 |
Reduce workaround code which I introduced for issue #17 Refactor TopBar::RefreshLayouts() to use efficient code.
Topbar is always failing to store layout path setting.
The text was updated successfully, but these errors were encountered: