Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • mutter mutter
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 1,157
    • Issues 1,157
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 124
    • Merge requests 124
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • GNOMEGNOME
  • muttermutter
  • Merge requests
  • !1543

wayland/text-input: Open the OSK after setting input focus

  • Review changes

  • Download
  • Email patches
  • Plain diff
Open Dor Askayo requested to merge doraskayo/mutter:text-input-enable into main Oct 30, 2020
  • Overview 8
  • Commits 1
  • Pipelines 4
  • Changes 1

From the commit message:

    wayland/text-input: Open the OSK after setting input focus
    
    Otherwise the OSK remains closed when only a single
    zwp_text_input_v3.enable request for a given surface is received from
    a client.
    
    The previous behavior required focusing (or pressing) text input boxes
    twice for the OSK to open.
    
    This undoes a part of eabb789381f6 which doesn't appear to be
    necessary to accomplish its purpose of keeping the OSK open on input
    focus change within a surface.

I tested this with gtk3-widget-factory, the reproducer provided in #1506 and a few GNOME apps. In every case the OSK seemed to open when text input was focused, close when it was unfocused, and remain open when switching between multiple text input boxes within the same surface. The behavior was similar in that regard to the behavior seen in the X11 session.

I also tested this with a touchscreen and everything appeared to work as expected.

Closes: #1506

CC: @carlosg, @jadahl

Edited Oct 30, 2020 by Dor Askayo
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: text-input-enable