Skip to content

Number keyboard support#440

Open
jbbjarnason wants to merge 13 commits intosony:masterfrom
centroid-is:number-keyboard-support
Open

Number keyboard support#440
jbbjarnason wants to merge 13 commits intosony:masterfrom
centroid-is:number-keyboard-support

Conversation

@jbbjarnason
Copy link

Propagate hint of keyboard, tested only v1 code path.


#include <functional>
#include <string>
#include <cstdint>
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

clang 20 was complaining of missing type name uint8_t

The DRM backend failed to translate between view property dimensions (specified
in scaled pixels) and Flutter window mentrics (specified in physical pixels).

This patch fixes two issues: first, given the DRM backend (questionably)
overrides the dimensions specified by the application with the native window
dimensions but neglects to compensate for the current scaling factor.

Secondly, cursor and touch events are not correctly mapped from physical pixels
to scaled pixels.

Signed-off-by: Luke Howard <lukeh@padl.com>
@martinetd
Copy link

Hi @jbbjarnason

This repo is being moved to https://github.com/flutter-elinux/flutter-embedded-linux -- would you mind re-opening your PR there?

Also, I'm afraid I'm not familiar with the text input protocol, I'll try to find time to check later but if you know of a way to test this that'd save me a lot of time and would be appreciated :)

martinetd and others added 8 commits January 29, 2026 14:05
These scripts are heavily based on what sony provided in [1],
with the container environment / cross compilation and some details
fixed up to allow rebuilding an older version more easily.

If these scripts live in the source repo further improvements can be
made to avoid cloning flutter-embedded-linux twice more, but they are
good enough for now, so let's start with what we have.

Link: sony/flutter-elinux#289 (comment) [1]
Build broke with gcc-15 due to `error: unknown type name 'uint8_t'`

Adding cstdint fixes that.
issues do not exist in flutter-elinux repo, so original sony links were kept

Also give thanks to sony for creating the fork
fix: add missing cstdint header
Fix DRM backend dimension scaling
flutter-elinux fork: update most URLs to github.com/flutter-elinux
@jbbjarnason
Copy link
Author

Hi @jbbjarnason

This repo is being moved to https://github.com/flutter-elinux/flutter-embedded-linux -- would you mind re-opening your PR there?

Also, I'm afraid I'm not familiar with the text input protocol, I'll try to find time to check later but if you know of a way to test this that'd save me a lot of time and would be appreciated :)

@martinetd I have copied this PR to the new fork

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.

4 participants