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

wl: Change IM context functions signature #671

Merged
merged 1 commit into from
Nov 23, 2023

Conversation

psaavedra
Copy link
Member

The input method context is now defined and assigned to the View once this is added to a Viewport, in cog_viewport_add(). It is not possible before because we need to have access to the wl_surface associated to the Viewport->Window for configuring the IM context.

@psaavedra psaavedra force-pushed the psaavedra/change_signature_im_context branch from ffaf9ee to 4faaba3 Compare November 23, 2023 10:35
@psaavedra psaavedra requested a review from aperezdc November 23, 2023 10:36
@psaavedra psaavedra self-assigned this Nov 23, 2023
@psaavedra psaavedra added the enhancement New feature or request label Nov 23, 2023
The input method context is now defined and assigned to the View once
this is added to a Viewport, in cog_viewport_add(). It is not possible
before because we need to have access to the wl_surface associated to
the Viewport->Window for configuring the IM context.
@psaavedra psaavedra force-pushed the psaavedra/change_signature_im_context branch from 4faaba3 to e25b826 Compare November 23, 2023 10:51
@psaavedra psaavedra enabled auto-merge (rebase) November 23, 2023 15:19
@psaavedra psaavedra merged commit f8d84ed into master Nov 23, 2023
@psaavedra psaavedra deleted the psaavedra/change_signature_im_context branch November 23, 2023 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants