-
-
Notifications
You must be signed in to change notification settings - Fork 109
Fancier icon-based workspace switcher #223
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
base: main
Are you sure you want to change the base?
Conversation
52d1380
to
7ca623f
Compare
7ca623f
to
9468da0
Compare
This version should be more robust |
I think it's broken after rebase, I've fixed it locally but will better test tomorrow. |
5ee7d8d
to
3e0b494
Compare
Please give it another try when you have time, it's really robust now, at least for me. |
When no label is used, it's not looking great (position are not matching the expectation). I suggest it could just be the same as the "icon" mode using " " as an icon, or something like this... I'm pushing a last commit to make things more consistent, but I didn't check anything vertical for now. |
Multi-monitor is still kind of broken but less than the current main branch in my case... I think it would require a bigger refactor regarding the event handling. |
WIP code, know problems:
Sorry for this PR ;)