keyboards icon indicating copy to clipboard operation
keyboards copied to clipboard

bug: Japanese picker keyboard is generating JS errors

Open keyman-sentry[bot] opened this issue 3 years ago • 4 comments

Sentry Issue: KEYMANWEB-COM-AX

TypeError: null is not an object (evaluating 'this.aq().parentNode')
  at None (/keyboard/japanese/1.0/japanese-1.0.js:1:1646)
  at None (/keyboard/japanese/1.0/japanese-1.0.js:1:864)
  at None (https://s.keyman.com/kmw/engine/14.0.265/src/keyboard.ts:59:14)
  at None (https://s.keyman.com/kmw/engine/14.0.265/src/kbdInterface.ts:984:21)
  at None (https://s.keyman.com/kmw/engine/14.0.265/src/keyboardProcessor.ts:225:25)
...
(5 additional frame(s) were not displayed)

As it's a handwritten keyboard, and legacy at that, fixing the underlying issue will likely require development team effort to resolve.

keyman-sentry[bot] avatar Mar 26 '21 01:03 keyman-sentry[bot]

Sentry issue: KEYMANWEB-COM-AY

keyman-sentry[bot] avatar Mar 26 '21 02:03 keyman-sentry[bot]

      /* Request a new candidates list (if context not empty) */
      this.getCandidates(hiragana);
      
      /* And display the KMW title bar, ensuring minimum width */
      KeymanWeb.ShowHelp();
>>>   this.getElem().parentNode.previousSibling.style.minWidth='14em';

Looks like this.getElem() can return null in some circumstances (this.elemID = 'Help_Keyboard_japanese').

  this.getElem = function()
  {
    this.elem = document.getElementById(this.elemID);
    return this.elem;
  };

Have not yet determined the circumstances under which the Help_Keyboard_japanese element can be missing.

mcdurdin avatar Mar 28 '21 21:03 mcdurdin