Page MenuHomePhabricator

Communication/coordination work: Selecting participant does not work correctly when using shift-selection
Closed, ResolvedPublic

Description

See T313077, if you try to shift-select participants, things won't be updated correctly. Since we cannot disable shift-selection for V0, we should communicate on this in T313077 with appropriate maintainers/contributors [who will we need to figure out].

Event Timeline

Change 814166 had a related patch set uploaded (by Daimona Eaytoy; author: Daimona Eaytoy):

[mediawiki/extensions/CampaignEvents@master] Disable shift-selection of participants on Special:MyEvents

https://gerrit.wikimedia.org/r/814166

Change 814166 abandoned by Daimona Eaytoy:

[mediawiki/extensions/CampaignEvents@master] Disable shift-selection of participants on Special:MyEvents

Reason:

https://gerrit.wikimedia.org/r/814166

Huh, checkboxShift.js only cares about classes on the input and CheckboxInputWidget does not let you set classes on the input. Aaaaaaaaalrighty then.

ifried renamed this task from Selecting participant does not work correctly when using shift-selection to Communication/coordination work: Selecting participant does not work correctly when using shift-selection.Jul 19 2022, 3:41 PM
ifried updated the task description. (Show Details)
ifried updated the task description. (Show Details)
ldelench_wmf added a subscriber: ifried.

Hey @ifried , I'm picking up this task to figure out who can help us - would you say the shift+select issue is a hard requirement for darkship? Or v1?

No, @ldelench_wmf, this isn't a hard requirement for V1. Thanks!

@ldelench_wmf I'm not sure what we should do with this task now. Should it be closed because the communicaton part is done? Or should we use it to track the specific bug in the CE extension? In any case, I think we need a task for our sprint board (mostly for QA).

Good call @Daimona , resolving this one - also pushed T313077 into our sprint board/code review.