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

[NUI][API10][AT-SPI] Set focus in OnAccessibilityActivated() #5387

Merged
merged 1 commit into from
Jul 12, 2023

Commits on Jul 11, 2023

  1. [NUI][AT-SPI] Set focus in OnAccessibilityActivated()

    This aligns the behaviour of Components (derived from CustomView), and other controls, with that
    of BaseComponents (and other controls with ViewAccessibilityMode.Default, backed by the C++
    accessibility implementation), which set the keyboard focus on activation. Please note that in the
    latter case, the implementation of View.OnAccessibilityActivated is unused, and the unrelated method
    (albeit bearing the same name) Toolkit::Control::OnAccessibilityActivated is called:
    
    bool Control::OnAccessibilityActivated()
    {
      if(Toolkit::KeyboardFocusManager::Get().SetCurrentFocusActor(Self()))
      {
        return OnKeyboardEnter();
      }
      return false;
    }
    
    So, in other words, this patch makes C# controls behave in a similar manner, i.e. set the keyboard
    focus on accessibility activation.
    Artur Świgoń committed Jul 11, 2023
    Configuration menu
    Copy the full SHA
    157e3b7 View commit details
    Browse the repository at this point in the history