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

Question: is the normal i3 (not i3-gaps) supported? #10

Closed
toku-sa-n opened this issue Nov 21, 2022 · 3 comments
Closed

Question: is the normal i3 (not i3-gaps) supported? #10

toku-sa-n opened this issue Nov 21, 2022 · 3 comments

Comments

@toku-sa-n
Copy link
Contributor

toku-sa-n commented Nov 21, 2022

Versions

%i3 --version
i3 version 4.20.1 (2021-11-03) © 2009 Michael Stapelberg and contributors

I've downloaded the prebuilt glimmer 0.0.2 from the release page.

Problem

I'm using the normal i3, not i3-gaps. When I try to use glimmer, I get the following error.

%./glimmer --styles ~/git_repository/glimmer/themes/example.css 
thread '<unnamed>' panicked at 'called `Result::unwrap()` on an `Err` value: Custom { kind: InvalidData, error: Error("unknown variant `machine`, expected one of `title`, `instance`, `class`, `window_role`, `transient_for`", line: 1, column: 727) }', src/actors/i3_ipc.rs:74:51
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
^C

Is this error due to not using i3-gaps? If it is, can I submit a PR to replace all the occurrences of the word i3 with i3-gaps to remove the confusion?

Reproduction

  1. Download the binary
  2. Run glimmer
  3. Change the window focus by moving the mouse cursor.

(BTW, i3-gaps is merged into i3 (PR). I think all the occurrences of i3-gaps should be replaced with "i3 with the gaps feature" or something.)

@moustacheful
Copy link
Owner

Hey again, sorry you've run into so many issues @toku-sa-n

i3 or i3-gaps should work, it might be the newer version of i3 that may be causing issues. Could you try compiling this branch #11 and let me know if works.

Thanks!

@toku-sa-n
Copy link
Contributor Author

It works! Thanks for the quick fix!

@moustacheful
Copy link
Owner

Glad to hear! thanks for reporting this.

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

No branches or pull requests

2 participants