Add Keyboard Editor to Management Console#871
Open
AdamRoden wants to merge 301 commits into
Open
Conversation
This may prevent spurious "missing .cab file" error that I once saw in the wild
- default to mouse - issue toast
These ffmpeg DLLs seem to be required by IrisbondHiruAPI.dll, bundling them in the app directly avoids need to install any Irisbond drivers for the Hiru
These are autotranslated, hopefully will get improved via transifex
This kept generating duplicate file which I was manually removing!
- Tweak UI on eye tracker selection page to include "other" info - Fix/update URL on eye tracker selection page - Add new static dialog for info about eye tracker support - Update synced files Fix (no) translations on eye tracker page We do not currently have translations available for this new static info dialog, so the strings have been made explicitly not-translatable Fix some missing translations on dialogs Fix URL on dialog For some reason this URL disappears when saved in the AI repository. This commit adds a replacement manually in each installer Fix newline in Symbol installer Fix translation of dialog title
There were some minor differences on my local branch which I'd been testing thoroughly, so this merge forces my changes onto the existing branch - Add static warning dialog re eyetracker deprecation - Some fixes to existing dialogs - Various strings and translations - Dealing with launching an app with a deprecated tracker - Bundle irisbond DLLs
This was causing a crash in management console
Member
|
@AdamRoden We are starting to work on a potential pre-release. Do you consider this PR ready for testing? |
Contributor
Author
|
@JuliusSweetland The PR is ready to test. |
Member
|
Great thanks. I have to prepare a pre release without this pr first to be
able to test and stabilise that work, but this will be integrated after
that. Have a good weekend.
…On Fri, 6 Jan 2023, 17:55 AdamRoden, ***@***.***> wrote:
@JuliusSweetland <https://github.com/JuliusSweetland> The PR is ready to
test.
—
Reply to this email directly, view it on GitHub
<#871 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABEA7AKMTCSKD736QPEEYXLWRBMBLANCNFSM6AAAAAARCQJEQ4>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Merged
Some apps were missing some setup. It's not all strictly required for all apps, but since we are fairly generous with exposing all the settings, it could easily be possible to configure an app to access more than intended, and trip up if resources haven't been set up.
These are no longer relevant with the new API. The setting remains in the settings but marked as obsolete. This commit removes references to it in the ViewModel
This came from rime-76a0a16-Windows-clang-x64\dist\lib We needed 64 bit DLL as Optikey is now 64 bit. The API used ints, which needed updating to UIntPtr throughout. No other changes seem to be necessary.
- Add various sub-options for pointsource depending on whether using touch or not - Fix Chinese coercion on language properties so it applies to all apps
- custom actions for new installer properties - XML file elements for config (new properties) - Chinese split language properties
I used the wrong property name.
new files, new date etc
+ move UI to fit
This lets us get out a chinese version with full translation.
We'll just make sure that the installer strings are translated
We were previous only using UI languages for installer translations, but want to include chinese keyboard languages too (mainly to cover chinese simplified)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
It took an overhaul of dynamic keyboards and the migration of various pieces of code to different classes in order for this to be built properly.
We should get many stakeholders to beta test this one to ensure their versions of keyboard files get interpreted as expected. I'm happy to test any keyboards shared with me if people submit them.
Let the review begin.