Skip to content
This repository has been archived by the owner on Jan 26, 2024. It is now read-only.

Display clients count in monocle symbol Fix #387 #388

Merged
merged 1 commit into from
Mar 16, 2023

Conversation

yveszoundi
Copy link
Contributor

  • Replicate missing functionality from dwl to display the client count in monocle mode
    • Add ltsymbol field to Monitor struct
    • Display client count in monocle mode when greater than zero
  • Tested with somebar and dwlb

@yveszoundi
Copy link
Contributor Author

This addresses issue #387

@yveszoundi
Copy link
Contributor Author

Please let me know if there are any concerns. In case it helps, I added few screenshots with most of the supported status bars.

Notes

  • I've not tried dtao as its configuration/usage is not "self-evident" to me, I vaguely recall trying out dzen a very long time ago. I assume that dtao consumes dwl printed output just like other bars.
  • For dwl-bar, I was never able to try it (empty bar on my system), not sure if it's related to my missing dependencies, musl, etc. I assume that it consumes dwl printed output just like other bars on machines that do render the bar properly.

Screenshots

somebar

somebar

dwlb

dwlb

yambar (latest git)

yambar

waybar

waybar

@MadcowOG
Copy link

MadcowOG commented Mar 6, 2023

A little unrelated and I don't mean to hijack. I'm sorry dwl-bar didn't work, I already had a commit made, but forgot to push it up. Everything should work now.

I tested and got a screenshot of dwl-bar working with your pr; I can't wait for this pr to be merged!

20230305-143307

@sevz17 sevz17 force-pushed the feature/387-monocle-client-count branch from 63b290a to 35e9611 Compare March 16, 2023 05:39
- Replicate missing functionality from dwl to display the client count in monocle mode
  - Add ltsymbol field to Monitor struct
  - Display client count in monocle mode when greater than zero
- Tested with somebar and dwlb
@sevz17 sevz17 force-pushed the feature/387-monocle-client-count branch from 35e9611 to bbdf2a9 Compare March 16, 2023 06:02
@sevz17 sevz17 merged commit bbdf2a9 into djpohly:main Mar 16, 2023
@sevz17
Copy link
Collaborator

sevz17 commented Mar 16, 2023

Thanks!

@yveszoundi yveszoundi deleted the feature/387-monocle-client-count branch March 16, 2023 22:29
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants