-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
Quotation mark #30481
base: main
Are you sure you want to change the base?
Quotation mark #30481
Conversation
@microsoft-github-policy-service agree |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for opening the PR. I've tried to test it, but it looks like VK_OEM_7 is not present in my keyboard. These look like they'll only be available in some keyboards, depending on language. Would it make sense to add it to a different key that's more prevalent? Wonder if .
makes sense, but I'm open to other ideas as well.
Perhaps the comma key, since it sort of looks like a quote? 🤷 |
I think |
src/modules/poweraccent/PowerAccentKeyboardService/KeyboardListener.h
Outdated
Show resolved
Hide resolved
src/modules/poweraccent/PowerAccentKeyboardService/KeyboardListener.idl
Outdated
Show resolved
Hide resolved
Hi. Is this pr ready? |
yes, i test it a last time. |
I suggest roll back the commit and use VK_QUOT_ |
Maybe move the mathematics symbols will be a better options. But i do not now on what key to put theme and if you are ok with this. I put the pull request back into draft. Because even if technically the code works, I am clearly not satisfied with the result. |
To give a little feedback. I'm thinking of working on character display. It seems that when there are too many characters, the display go out of screen.
Sorry it's taking so long for this pool request |
I want to point out that the display (when put on all the languages) go still outisde the screen.
I added focus on the curent char via `ScrollIntoView' when the user changes the active char. |
Miss click. I defined the maximum width that the window could have. |
@Jay-o-Way The pr is ready. |
Add Quotation mark
Add local quotation based on VK_OEM_7(0xDE) key.
Not all quotes have been added, only
‟ „ ” « » ‚ , ‘ ’ › ‹ '「 」 《 》 『 』〈 〉
Why not added :
⹂ ⌜ ⌝ ❛ ❜ ❝ ❞ 🙶 🙷 🙸 ' 「 」
its redundant and would make too much and not readable.﹁ ﹂ ﹃ ﹄
I did not put them because there use for horizontal textPR Checklist
This PR is currently a draft, I still need to know if adding language-related keyboard management is a good idea or specifying the use of a gobal key to make it easier to manage all the keyboards in one. Some languages can use different keyboards, I think this would become problematic if the keyboard does not match the key used by default. However, using a universal key can also pose an issue to finding the key. that remains to be discussed
Validation Steps Performed