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

HMI sends redundant UI.OnKeyboardInput 'INPUT_KEY_MASK_ENABLED' on press 'KEYBOARD' button #529

Closed
VjKlepikov opened this issue Mar 22, 2021 · 3 comments
Labels
5.5.0 Release issues introduced by 5.5.0 release candidate

Comments

@VjKlepikov
Copy link
Contributor

Description:
HMI sends redundant UI.OnKeyboardInput 'INPUT_KEY_MASK_ENABLED' on press 'KEYBOARD' button

Precondition:

  1. Default app_policies groups is updated in sdl_preloaded_pt.json
    "app_policies": {
    "default": { ...
    "groups": [
    "Base-4", "OnKeyboardInputOnlyGroup"
    ],
    ... }}
  2. SDL and HMI are started
  3. Apps is registered and activated
  4. App sends SetGlobalProperties RPC with (maskInputCharacters: "USER_CHOICE_INPUT_KEY_MASK")

Steps to reproduce:

  1. Press "KEYBOARD" button on HMI

Expected result:

  1. HMI does not send OnKeyboardInput notification

Actual result:

  1. HMI sends UI.OnKeyboardInput{"event":"INPUT_KEY_MASK_ENABLED"} after receiving UI.SetGlobalProperties request

Environment:
SDL: https://github.com/smartdevicelink/sdl_core/tree/develop (0df6608)
HMI: https://github.com/smartdevicelink/sdl_hmi/tree/develop (7fad546)
SPT: 20210305-Android (1076)

@VjKlepikov
Copy link
Contributor Author

@jordynmackool, Please notice this issue is related to feature Keyboard Enhancements that should be released in the 7.1 release.

@jordynmackool jordynmackool added the 5.5.0 Release issues introduced by 5.5.0 release candidate label Mar 22, 2021
@VjKlepikov
Copy link
Contributor Author

@JackLivio Please note that the issue is reproduced on 5.5.0 Release in case:

Precondition: the same

Steps to reproduce:

  1. Press 'unmask' key on HMI keyboard.
  2. Press "KEYBOARD" button on HMI

Expected result:

  1. HMI→SDL: "UI.OnKeyboardInput": ({{"event":"INPUT_KEY_MASK_DISABLED"}}})
    SDL→App: "OnKeyboardInput": ({{"event":"INPUT_KEY_MASK_DISABLED"}}})
  2. HMI does not send OnKeyboardInput notification

Actual result:

  1. HMI→SDL: "UI.OnKeyboardInput": ({{"event":"INPUT_KEY_MASK_DISABLED"}}})
    SDL→App: "OnKeyboardInput": ({{"event":"INPUT_KEY_MASK_DISABLED"}}})

  2. HMI sends UI.OnKeyboardInput
    {"event":"INPUT_KEY_MASK_ENABLED"}
    after receiving UI.SetGlobalProperties request

Environment:
SDL: https://github.com/smartdevicelink/sdl_core/tree/7.1.0-RC1
HMI: https://github.com/smartdevicelink/sdl_hmi/commits/release/5.5.0 (b1d31f8)
SPT: SPT: 20210305-Android (1076)

@Jack-Byrne
Copy link
Collaborator

@VjKlepikov I dont believe this is an issue. When the mask mode is USER_CHOICE_INPUT_KEY_MASK, every time the keyboard is presented it will by default have masking enabled.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5.5.0 Release issues introduced by 5.5.0 release candidate
Projects
None yet
Development

No branches or pull requests

3 participants