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

dwl/tags: add hide-vacant option #3925

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

casKd-dev
Copy link

Equivalent to hide-vacant-tags patch for dwm, hiding non-active tags without clients

@DreamMaoMao
Copy link
Contributor

#3504

Is this the same function as this pr?

@casKd-dev
Copy link
Author

#3504

Is this the same function as this pr?

Indeed, it is even better. I've submitted these as a batch after writing them for my own use, so i wasn't aware it already was proposed.

@casKd-dev casKd-dev closed this Mar 24, 2025
@DreamMaoMao
Copy link
Contributor

#3504

This pr seems to have been abandoned, It seemed that no further work had been done for a long time,you should continue your pr, this is a good feature.

@casKd-dev casKd-dev reopened this Mar 24, 2025
@DreamMaoMao
Copy link
Contributor

DreamMaoMao commented Mar 24, 2025

it didn't have the apply hidden when waybar started up.

@DreamMaoMao
Copy link
Contributor

DreamMaoMao commented Mar 24, 2025

I think you should not add all the buttons at first, but check if the current tag button is already added in the handle_view_tags, and add it if it is not

@DreamMaoMao
Copy link
Contributor

Sorry I couldn't wait, so I made my own...

#4003

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

Successfully merging this pull request may close these issues.

2 participants