Scoped labels: don't require holding alt key to remove (#23303) (#23331)

Backport #23303

Alt doesn't work on all browsers, the simplest solution for v1.19 is to
just not require it and toggle the label by just clicking.

Part of #22974

Co-authored-by: Brecht Van Lommel <brecht@blender.org>
This commit is contained in:
Giteabot 2023-03-06 13:59:53 -05:00 committed by GitHub
parent 233a399706
commit af4767df5c
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -145,7 +145,6 @@ export function initRepoCommentForm() {
const clickedItem = $(this); const clickedItem = $(this);
const scope = $(this).attr('data-scope'); const scope = $(this).attr('data-scope');
const canRemoveScope = e.altKey;
$(this).parent().find('.item').each(function () { $(this).parent().find('.item').each(function () {
if (scope) { if (scope) {
@ -153,11 +152,7 @@ export function initRepoCommentForm() {
if ($(this).attr('data-scope') !== scope) { if ($(this).attr('data-scope') !== scope) {
return true; return true;
} }
if ($(this).is(clickedItem)) { if (!$(this).is(clickedItem) && !$(this).hasClass('checked')) {
if (!canRemoveScope && $(this).hasClass('checked')) {
return true;
}
} else if (!$(this).hasClass('checked')) {
return true; return true;
} }
} else if (!$(this).is(clickedItem)) { } else if (!$(this).is(clickedItem)) {