The text will be cleared on codemirror with the Japanese Kanji input

[miyauchi]

If we entered a Japanese character, the text will be cleared.
We can see this problem on Custom HTML widget and editor screens for plugin and themes,

[ryotsun]

Replying to miyauchi:

If we entered a Japanese character, the text will be cleared.
We can see this problem on Custom HTML widget and editor screens for plugin and themes,

It seems depending on editor settings of codemirror below.
https://core.trac.wordpress.org/browser/trunk/src/wp-includes/general-template.php#L3207

contenteditable should be textarea to fix this issue.

        $settings = array(
                'codemirror' => array(
                        ...
                        'inputStyle'       => 'textarea',
                        ...
                ),
                ...
        );

But I need some assistance because I'm not sure expected impact area.
(Impact area I found so far is not only customHTML widget but also "Theme Editor".)

cf.) ​https://codemirror.net/doc/manual.html

inputStyle: string

Selects the way CodeMirror handles input and focus. The core library defines the "textarea" and "contenteditable" input models. On mobile browsers, the default is "contenteditable". On desktop browsers, the default is "textarea". Support for IME and screen readers is better in the "contenteditable" model. The intention is to make it the default on modern desktop browsers in the future.

related issue

it may the related issue..?
​https://github.com/codemirror/CodeMirror/issues/3137

[miyauchi]

@ryotsun

I tried your patch and it looks good to me.

But I found following in the codemirror documentation:

inputStyle: string
Selects the way CodeMirror handles input and focus. The core library defines the "textarea" and "contenteditable" input models. On mobile browsers, the default is "contenteditable". On desktop browsers, the default is "textarea". Support for IME and screen readers is better in the "contenteditable" model. The intention is to make it the default on modern desktop browsers in the future.

​https://codemirror.net/doc/manual.html

I am not sure but I think it should be same behavior with the codemirror default.
So, it shoud be contenteditable on mobile and it should be textarea on desktop.(?)

[ryotsun]

@miyauchi

I am not sure but I think it should be same behavior with the codemirror default.

I agree with you. If we follow its default, it doesn’t have to set inputStyle.
But in this case, it’s a bit hard to do so because there is a known issue on mobile.
cf.)
​https://github.com/codemirror/CodeMirror/issues/3137

that’s why, I guess to set textarea is a better solution for now.

[pento]

According to the CodeMirror issue, this appears to be a bug that won't be solvable until CodeMirror 6 is released, which is still some time away.

We'll certainly upgrade to CodeMirror 6 when it's released, but I don't think there's anything we can do right now.