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

Support glow with sctk using updated version of iced_glutin #30

Draft
wants to merge 58 commits into
base: master
Choose a base branch
from

Conversation

ids1024
Copy link
Member

@ids1024 ids1024 commented Feb 2, 2023

Based on iced-rs#1688, which still may need improvements to config handling and errors but mostly works.

Still need to test this with libcosmic, and support this in dyrend.

mmstick and others added 30 commits December 19, 2022 16:13
This reverts commit f1ada7a.

Fixes panic when using lazy widgets
feat: various improvements to window commands and event handling
* sctk: Don't send output events twice

With these lines, a subscription seems to receive output events twice.
Without it, only once as expected.

* sctk: Make `IcedOutput::Output` take a `WlOutput`

This is the most flexible solution, and using output events, it is
possible to create surfaces for specific outputs or all.

Matching by make/model is a bad idea anyway. Matching by name may be
appropriate.
Drakulix and others added 23 commits January 4, 2023 21:03
…aces, and also adds the option for starting with no initial surface
For some reason this was an issue testing with `iced_glow`/`glutin`, but
this also fixes the issue I was having where the application doesn't get
output events if I destroy the initial surface and create surfaces
later.
This provides a `Compositor` adapting a `GLCompositor`, that thus can be
used with `iced_winit` or with a different shell.

Backport of iced-rs#1688. Still needs some
work.
Fixes behavior with `InitialSurface::None`.
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.

6 participants