Fossil SCM
chat: seem to have eliminated the tiny dead-zone between the label and checkbox in the settings menu.
Commit
dc8f3a3692628ca342695ecf2398d94a7c42d3061d220c6d6662360e1c5c5217
Parent
1221a579ecdbf4d…
1 file changed
+1
-2
+1
-2
| --- src/chat.js | ||
| +++ src/chat.js | ||
| @@ -830,14 +830,13 @@ | ||
| 830 | 830 | D.append(line, btn); |
| 831 | 831 | if(op.hasOwnProperty('boolValue')){ |
| 832 | 832 | const check = D.attr(D.checkbox(1, op.boolValue()), |
| 833 | 833 | 'aria-label', op.label); |
| 834 | 834 | D.append(line, check); |
| 835 | - check.addEventListener('click', callback); | |
| 836 | 835 | } |
| 837 | 836 | D.append(settingsPopup.e, line); |
| 838 | - btn.addEventListener('click', callback); | |
| 837 | + line.addEventListener('click', callback); | |
| 839 | 838 | }); |
| 840 | 839 | }; |
| 841 | 840 | settingsPopup.installHideHandlers(false, true, true) |
| 842 | 841 | /** Reminder: click-to-hide interferes with "?" embedded within |
| 843 | 842 | the popup, so cannot be used together with those. Enabling |
| 844 | 843 |
| --- src/chat.js | |
| +++ src/chat.js | |
| @@ -830,14 +830,13 @@ | |
| 830 | D.append(line, btn); |
| 831 | if(op.hasOwnProperty('boolValue')){ |
| 832 | const check = D.attr(D.checkbox(1, op.boolValue()), |
| 833 | 'aria-label', op.label); |
| 834 | D.append(line, check); |
| 835 | check.addEventListener('click', callback); |
| 836 | } |
| 837 | D.append(settingsPopup.e, line); |
| 838 | btn.addEventListener('click', callback); |
| 839 | }); |
| 840 | }; |
| 841 | settingsPopup.installHideHandlers(false, true, true) |
| 842 | /** Reminder: click-to-hide interferes with "?" embedded within |
| 843 | the popup, so cannot be used together with those. Enabling |
| 844 |
| --- src/chat.js | |
| +++ src/chat.js | |
| @@ -830,14 +830,13 @@ | |
| 830 | D.append(line, btn); |
| 831 | if(op.hasOwnProperty('boolValue')){ |
| 832 | const check = D.attr(D.checkbox(1, op.boolValue()), |
| 833 | 'aria-label', op.label); |
| 834 | D.append(line, check); |
| 835 | } |
| 836 | D.append(settingsPopup.e, line); |
| 837 | line.addEventListener('click', callback); |
| 838 | }); |
| 839 | }; |
| 840 | settingsPopup.installHideHandlers(false, true, true) |
| 841 | /** Reminder: click-to-hide interferes with "?" embedded within |
| 842 | the popup, so cannot be used together with those. Enabling |
| 843 |