Docdiff should not apply the 'd' hotkey in textareas

[davidfischer]

Docdiff doesn't apply the d hotkey when typing in <input> fields but should apply similar checks to <textarea> fields.

addons/src/hotkeys.js

Lines 49 to 69 in ff8baf2

	_handleKeydown = (e) => {
	// Close docdiff with single-stroke `d` (no Ctrl, no Shift, no Alt and no Meta)
	// (I'm checking `document.activeElement` to check if it not inside an INPUT to avoid enable/disable while typing on forms)
	// Read more about these decisions at https://github.com/readthedocs/addons/issues/80
	let event;
	// DocDiff
	if (
	this.docDiffHotKeyEnabled &&
	keyboardEventToString(e) ===
	this.config.addons.hotkeys.doc_diff.trigger &&
	document.activeElement.tagName !== "INPUT"
	) {
	if (this.docDiffShowed) {
	event = new CustomEvent(EVENT_READTHEDOCS_DOCDIFF_HIDE);
	this.docDiffShowed = false;
	} else {
	event = new CustomEvent(EVENT_READTHEDOCS_DOCDIFF_ADDED_REMOVED_SHOW);
	this.docDiffShowed = true;
	}
	}

You can see this bug in the notes field on ethicalads.io where the 'd' character cannot be typed.

[astrojuanlu]

We're still spotting this kedro-org/kedro#3371 (comment)

[humitos]

@astrojuanlu thanks for commenting here pointing out to a reproducible example. I think the issue you are hitting is similar to the one this issue mentions and it was solved in v0.9.0 by this PR of the addons which I released the past Tuesday but I had to revert that deploy because it broke other stuffs 😅 (see #207)

I double checked the code and it seems it's correct: https://github.com/readthedocs/addons/blob/main/src/hotkeys.js#L58-L63

Once I fix the 0.9.0, I will do another one that once deployed it should solve the issue you are experimenting. I'm targeting this release to next Tuesday. If you want, I can disable the "hotkeys addons" in your project now as an immediate workaround. Let me know.

I'll keep you posted otherwise 👍🏼

[astrojuanlu]

Thanks! cc @rashidakanchwala