pmusaraj
21 أبريل 2023، 6:34م
15
شكرًا، يجب إصلاح هذا عبر:
main ← search-selection-tweak
opened 05:37PM - 21 Apr 23 UTC
Followup to 17ba00c3950eb0a78dd510f3618f8e7be2d448fc. Fix for https://meta.disco… urse.org/t/-/261917
This fixes a usability issue where the user couldn't switch to the user menu when the search menu was visible and the text in the input was selected.
Explanation: The `click` event is triggered both when clicking and when selecting text. This means that when selecting text in the search input, at the end of the selection event, a click event is triggered. And if that click event happens outside of the search menu (i.e. when you continue dragging outside the input), then the menu would be dismissed.
Previously, we fixed this by checked for the presence of a current text selection. But that results in a small side-effect reported in the linked issue on meta. This PR sets a flag during `mouseDown` when starting a text selection in the input and then it uses that flag when evaluating whether to trigger `clickOutside` or not.
يوجد شرح طويل إلى حد ما في وصف طلب السحب (PR)، النسخة المختصرة هي أننا كنا نتجاهل سابقًا حدث clickOutside لقائمة البحث عندما كان هناك نص محدد. لكننا نحدد تلقائيًا نص قائمة البحث، لذا يتحول هذا إلى تقنية مختلفة.
4 إعجابات