Skip to content

Draft: Feat/sway#7

Open
nardoor wants to merge 3 commits intomoustacheful:masterfrom
nardoor:feat/sway
Open

Draft: Feat/sway#7
nardoor wants to merge 3 commits intomoustacheful:masterfrom
nardoor:feat/sway

Conversation

@nardoor
Copy link

@nardoor nardoor commented Nov 16, 2022

This is my go at adding sway compatibility.
This shouldn't break anything on i3.

  • I updated the dependencies
  • Added a --use-sway option (probably some way to detect automatically)
  • I used swayipc to do the equivalent of i3_ipc
  • I wrapped their different (but quite similar) types for the rest of the program to run similarly.

There is at least one bug I am aware of, there are probably more :
On sway + weston + xwayland,

  • the popup windows steal focus
  • because they steal focus, when they die, the focus changes and a popup window spawns again

I am unsure how to solve this, even less how to investigate, this might be a xwayland bug or a weston bug or something else.

Please let me know your feedback, thoughts and ideas.
Have a nice day.

@its-darsh
Copy link

is this draft is ready or its just dead?

@nardoor
Copy link
Author

nardoor commented Oct 24, 2023

@its-darsh currently not working on it

We could say dead for now, but if you have suggestions I'd take it.

I could try updating the dependencies to see if the bugs I faced are gone by themself.

@markstos
Copy link

markstos commented Nov 5, 2023

I would consider using this with Sway if it worked.

@its-darsh
Copy link

i'm actually interested to discuss the facing issues for the project not to work under sway

i just want some way to communicate with you all, discord or github discussions will be the best way for me

@nardoor
Copy link
Author

nardoor commented Nov 17, 2023

I opened a discussion there nardoor#1
Are you able to access it ?

@its-darsh
Copy link

I opened a discussion there nardoor#1 Are you able to access it ?

yes i am, continuing there

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.

3 participants