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

[xkeyboard-config] Map Xperia 10 III assistant button to camera. JB#60142 #3

Merged
merged 1 commit into from Feb 13, 2023

Conversation

pvuorela
Copy link
Contributor

Copy link

@Thaodan Thaodan left a comment

Choose a reason for hiding this comment

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

Same here, the button is probably shared between different ports.

@pvuorela
Copy link
Contributor Author

Same here, the button is probably shared between different ports.

Yes, and that's ok. We have been just using one mapping to all the devices we've supported. Actually this being the jolla phone mapping here. In theory there could be a collision on the custom keycodes between devices, but in practice new ones are encountered only every few years. And if we'd have different keymaps, we'd need to get more support on configuring one for each device.

With a high likelihood of never getting a collision we can just postpone these complication. Thus won't make changes here.

Copy link

@spiiroin spiiroin left a comment

Choose a reason for hiding this comment

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

But neither the change + associated comment nor the git commit message exactly spells out what is happening / what these numbers are. But that might be just me being unfamiliar with the files in here.

I guess this is all right and turns assistant key to camera key in every device out there - which would be fine by me for now.

…0142

At least Xperia 10 III sends a custom 0x01C9 on the assistant key
which is at the same place and in the same form as other devices have
a camera key. Thus handling that as camera key now as there is no
assistant related functionality here. Nor even clear alternative
symbol to map into.
@pvuorela
Copy link
Contributor Author

Added a few words to the commit message.

@pvuorela pvuorela merged commit 4e6a7d6 into master Feb 13, 2023
@rainemak rainemak deleted the assistant_button branch March 13, 2023 11:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants