Skip to content
This repository has been archived by the owner on Nov 1, 2021. It is now read-only.

data-device: allow multiple devices for the same seat #1384

Merged
merged 2 commits into from
Nov 26, 2018

Conversation

emersion
Copy link
Member

@emersion emersion commented Nov 21, 2018

This commit makes it possible for a single client to have multiple data devices
for the same seat. This fixes issues with Firefox.

This mainly removes wlr_data_source.offer. We make sure we create one data
offer per device. We now make the offer inert when the source is destroyed.

Fixes #1041 (the second half)

@uwqwxing886
Copy link

This fixes the Firefox crash when pasting the clipboard contents into the address bar. Is there anything else that should be tested?

Firefox is still laggy but that's an issue with the Firefox wayland port itself.

@ghost
Copy link

ghost commented Nov 24, 2018

Works fine, this is the last thing needed to make firefox with wayland work here.

@cg9999
Copy link

cg9999 commented Nov 25, 2018

With this fix, firefox is now perfectly usable. The lag issue has also been fixed by other commits.
Only minor issue left for me seems to be some redraws missed behind closing menus and tooltips.

@emersion emersion changed the title data-device: allow multiple devices for the same seat [WIP] data-device: allow multiple devices for the same seat Nov 25, 2018
@emersion
Copy link
Member Author

This breaks copy-pasting in gnome-terminal. To reproduce:

pkill gnome-terminal
/usr/lib/gnome-terminal-server
gnome-terminal; gnome-terminal

And try copy-pasting between the two terminal windows.

@emersion emersion changed the title [WIP] data-device: allow multiple devices for the same seat data-device: allow multiple devices for the same seat Nov 26, 2018
@emersion
Copy link
Member Author

Fixed, this is ready for review.

This commit makes it possible for a single client to have multiple data devices
for the same seat. This fixes issues with Firefox.

This mainly removes wlr_data_source.offer. We make sure we create one data
offer per device. We now make the offer inert when the source is destroyed.

Fixes the second half of swaywm#1041
The source could be used in another offer.
@ghost
Copy link

ghost commented Nov 26, 2018

Works fine now.

@ddevault ddevault merged commit a2b8d47 into swaywm:master Nov 26, 2018
@ddevault
Copy link
Contributor

Thanks!

@emersion emersion deleted the data-multiple-devices branch January 18, 2019 13:06
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Firefox-wayland crash when selection is available or clipboard has contents
4 participants