redmica / redmica_ui_extension

This plugin adds useful UI improvements to RedMica.
GNU General Public License v2.0
41 stars 12 forks source link

Changed to trigger the change event twice only when there is a native change event #30

Closed ishikawa999 closed 2 years ago

ishikawa999 commented 2 years ago

17

問題の内容:

今回の変更: 根本的な問題の解決は難しいため、ネイティブなイベントが存在するときにのみイベントを2回発生させるように変更します。 これによって、ネイティブなイベントが存在しないときは通常通りの動きになるためイベントが2回発生することによる多くの問題は解決するはずです。

ネイティブなイベントが存在することを取得する方法は関数として提供されていないため、 EventTarget.prototype.addEventListener を上書きし、addEventListenerが動いたという情報を追加するdata属性をelementに対して追加するようにしています。

@yui-har レビューをお願いいたします。

yui-har commented 2 years ago

@ishikawa999 masterブランチにマージして良いと思います。

ishikawa999 commented 2 years ago

@ishikawa999 masterブランチにマージして良いと思います。

ご確認いただきありがとうございました。マージします。