-
-
Notifications
You must be signed in to change notification settings - Fork 707
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
Feature: Group Drawers #2574
Feature: Group Drawers #2574
Conversation
fix: typo in handleMouseHover method name
Linter failed because of an unrelated file. This file has been linted in #2563, which I believe will be merged first, so I won't be bothering to do it on this PR as well. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested locally and works great, another cool feature! Thanks.
One thing I've been wanting is this functionality for the workspaces with the window-rewrite functionality. It would be nice to have the workspace name / icon always shown and then hover over the workspace to see the window-rewrite icons inside it slide out. |
That's a pretty neat idea! I'm just finishing another feature and as soon as I open its PR I'll see if I can add a parameter to allow for that! |
@Alexays this one should be good too |
Love it! |
Just added the documentation to the wiki! |
"custom/power", // First element is the "group leader" and won't ever be hidden | ||
"custom/quit", | ||
"custom/lock", | ||
"custom/reboot", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is the comma required on this line? I guess no.
@Syndelis could you please share your waybar's |
Aside from the config listed in the PR, I believe there's just this CSS. Is there anything in particular that you couldn't get working? @define-color macchiato-blue #8aadf4;
@define-color macchiato-yellow #eed49f;
@define-color macchiato-mauve #c6a0f6;
#group-power {
margin-bottom: 0px;
}
#group-power .text-button {
font-weight: normal;
}
#group-power .drawer .text-button {
padding-bottom: 24px;
}
#custom-power { color: @macchiato-red; padding-bottom: 6px; }
#custom-reboot { color: @macchiato-yellow; }
#custom-lock { color: @macchiato-blue; }
#custom-quit { color: @macchiato-mauve; } |
Great, I didn't get that I had to use the following classes to address hidden items. Thank you.
|
This reverts commit de84626. With the new features [0][1] in waybar I feel I no longer need menubar, volume slider and audio buttons in swaync. KISS [0] Alexays/Waybar#2574 [1] Alexays/Waybar#2575
Is it possible to add an option to expand to the right or left? |
Yeah I was looking for this functionality too. But, forgot to request it. |
I'm sorry, but isn't what
|
it's just a sliding effect |
Gotcha, luckily I only used groups on the right-located modules. Hence, I didn't experience such a need. I agree with you that it would be nice to have. @Syndelis what do you think? |
I'll take a second look at adding this; might take a while because GTK's documentation is pretty bad |
I've taken an initial look at this (the same as the above) but unfortunately I'm having little time to contribute right now as I'm running close to my master's thesis' deadline. Being more realistically I'll probably only get back to it in early 2024. I'll gladly review any attempts at this, however! |
I have came across an issue where the group drawer display correctly, yet once hover over the drawer elements (apart from parent icon) the waybar freezes. -l debug gives no errors or warnings. Tested on different device with similar setup and the issue wasn't there. Both machines have same kernel, same waybar build. Tried default config for the drawer from Wiki, same issue on one machine, no problem on another. Does the drawer function add any dependencies by any chance? |
Is this built from latest on master? A distribution package manager? |
Also, how does your bar config look? Maybe your elements have a |
@khaneliman built from latest master by yay (AUR), package waybar-cava v0.9.24-1 - both machines have same build, same config, via same method. config of the group/drawer:
|
Could you try using the regular waybar package? Oh, and let's move this conversation to an issue, doesn't make much sense to discuss this here |
Tried as well, same behavior. Works on PC, doesn't on laptop. Created an issue thread - #2845 |
Any news about this feature? |
For my part I'm still busy with my master's which is due late march / early april. Until then I'm only working on bug fixes and the like for Waybar. Once again, I'll be glad to review any PRs that do attempt to introduce this! In the meanwhile, could any of you open an issue for this feature request? We shouldn't be using this closed PR to track this suggestion/request. |
About this PR
This PR introduces "group drawers". This feature allows to user to hide all but the first element of a group until the mouse hovers over the group. The user can configure the animation's speed and direction.
Demo Video
waybar_drawer.mp4
Configuration
The configuration to enable group drawers is available as a key under a group: