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

[Tech Debt] Reduce using qt private classes: QKeyMapper #23988

Open
2 tasks done
Eism opened this issue Aug 12, 2024 · 0 comments
Open
2 tasks done

[Tech Debt] Reduce using qt private classes: QKeyMapper #23988

Eism opened this issue Aug 12, 2024 · 0 comments
Labels
feature request Used to suggest improvements or new capabilities tech debt

Comments

@Eism
Copy link
Contributor

Eism commented Aug 12, 2024

Your idea

We use the private class QKeyMapper for navigation using symbols in the menu for Windows and Linux. This approach can lead to backward compatibility issues, for example here and here

QKeyMapper is needed to solve the case when the user has switched the keyboard layout to a non-English one and is trying to navigate. A similar problem was solved for shortcuts in MacOS and QKeyMapper was used there before, there too, a decision was made to abandon QKeyMapper - to use native platform tools to solve the problem of translating symbols between layouts.

Most likely, we need to create a KeyMapper class and implement it for all platforms.

Problem to be solved

Reduced using qt private classes

Prior art

No response

Additional context

No response

Checklist

  • This request follows the guidelines for reporting issues
  • I have verified that this feature request has not been logged before, by searching the issue tracker for similar requests
@Eism Eism added the tech debt label Aug 12, 2024
@muse-bot muse-bot added the feature request Used to suggest improvements or new capabilities label Aug 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Used to suggest improvements or new capabilities tech debt
Projects
None yet
Development

No branches or pull requests

2 participants