Skip to content

feat(i3status): new applet to embed i3status in panel/dock #344

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

Closed
wants to merge 3 commits into from

Conversation

jokeyrhyme
Copy link
Contributor

@jokeyrhyme jokeyrhyme commented Apr 13, 2024

  • this PR adds a new applet that serves as an adapter for processes that implement the i3bar protocol ( https://i3wm.org/docs/i3bar-protocol.html )
  • this might make COSMIC more desirable for folks who are coming from i3/sway and already have their bar setup exactly the way they like it
  • it could also serve as an alternative path for applet development, for folks who aren't yet comfortable with Rust but are able to produce JSON streams, etc

my example i3status-rust setup running in swaybar:
out

the same example running within this new COSMIC applet:
screenshot-2024-04-13-02-58-58

must-haves:

  • launch a real i3status-rs process and subscribe to its stdout
  • parse the stdout lines into COSMIC text widgets
  • bike-shed a much better name (I3status in code irritates me so much, haha)
  • applet icon
  • sort out error handling
  • use the same font/size as the time/date applet
  • support auto-detection of i3status and i3status-rs executables
  • support basename-only commands by searching PATH (using which ?)

follow-up items (future PRs):

  • provide a GUI for users to configure the font/size
  • figure out what to do with pango-versus-nomarkup
  • provide a GUI for users to add their own preferred command (e.g. a custom executable built with barista)
  • feed user clicks through to i3status process

@jokeyrhyme jokeyrhyme marked this pull request as draft April 13, 2024 04:51
@jokeyrhyme
Copy link
Contributor Author

Bahaha, okay, I just noticed #92 which may be more sophisticated than my effort, but I take this as validation that I'm not the only person on Earth who wants this feature to exist :)

@ryanabx
Copy link
Contributor

ryanabx commented Apr 13, 2024

Bahaha, okay, I just noticed #92 which may be more sophisticated than my effort, but I take this as validation that I'm not the only person on Earth who wants this feature to exist :)

That PR is pretty old, at the very least it's good to have an up-to-date version of it. If it looks like Ian's version is more robust though, maybe you could rebase his PR to get it up to date.

@jokeyrhyme
Copy link
Contributor Author

Yep, I finally have a few hours to spare, so I'm going to branch from the other PR (to preserve the authorship of those commits) and then work through the checklist I have in this PR and see where I end up

@jokeyrhyme
Copy link
Contributor Author

Closed in favour of #360

@jokeyrhyme jokeyrhyme closed this Apr 19, 2024
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