emacs_chrome icon indicating copy to clipboard operation
emacs_chrome copied to clipboard

various cleanups while tracking down an issue

Open attila-lendvai opened this issue 3 years ago • 2 comments

various cleanups while tracking down why it doesn't work on https://www.w3schools.com/tags/tryit.asp?filename=tryhtml_textarea

attila-lendvai avatar Jul 14 '22 11:07 attila-lendvai

What is this PR trying to achieve? I don't really want to merge a bunch of ancillary code motions and debug logs unless they are going to be useful in the long term. What is the actual fix in here?

stsquad avatar Jul 14 '22 16:07 stsquad

What is this PR trying to achieve? I don't really want to merge a bunch of ancillary code motions and debug logs unless they are going to be useful in the long term. What is the actual fix in here?

it fixes nothing. what it does is it greatly improves debuggability.

the issue that needs to be fixed is that the contextmenu event is not triggered at various places. e.g. see the link in the PR description.

attila-lendvai avatar Jul 15 '22 07:07 attila-lendvai