feat(tree-select): multiselect without checkbox + multiselect with shift and ctrl (#UIM-156) #400
Conversation
|
Preview docs changes for 7b6de50 at https://positive-js.github.io/mosaic-previews/pr400-7b6de50/ |
|
Preview docs changes for 940c82d at https://positive-js.github.io/mosaic-previews/pr400-940c82d/ |
|
Preview docs changes for a9da5c6 at https://positive-js.github.io/mosaic-previews/pr400-a9da5c6/ |
There was a problem hiding this comment.
Ctrl + клики мышкой проверить не могу, под маком не работает. Проверил с клавиатурой.
Проверял все на https://positive-js.github.io/mosaic-previews/pr400-a9da5c6/list/overview,
Ошибка 0.
http://screenshots.ptsecurity.com/mozornin-2020-02-10_19-53-03.png
Не работает мультиселект
Первый пример Multiple mode without checkboxes
Ошибка 1.
- Выбрать элемент Item 2
- Нажать shift, кликнуть по Item 8
Что не так: не только выбираются элементы, но и выделяется текст http://screenshots.ptsecurity.com/mozornin-2020-02-10_19-42-27.png
Ошибка 2.
- Выбрать элемент Item 2
- Зажать shift, нажать PgDn
- Выберутся элементы ниже (правильно)
- Не отжимая shift нажать PgUp.
- Выберутся элементы выше (правильно), но при этом останутся выбраными элементы ниже (неправильно).
При навигации без PgDn просто курсором ↓↑ поведение корректное.
Ошибка 3 (полное видео: http://d.pr/zxMQrl)
- Выбрать с ctrl элементы Item 2, Item 4, Item 6
- Не отжимая Ctrl перейти фокусом на элемент Item 10.
- Отжать ctrl
- Зажать shift, начать двигаться вверх. Элементы начнут выбираться
- Элементы 7, 5 и 3 после прохождения курсора развыбираются (неправильно, не должны)
Ошибка 4
- Выбрать с ctrl элемент Item 2, Item 3, Item 4, Item 7
- Не отжимая Ctrl перейти фокусом на элемент Item 2.
- Отжать ctrl
- Зажать shift, начать двигаться вверх. Элементы начнут развыбираться (правильно)
- Элемент 4 не развыбирается, элемент 5 и 6 выбирается (не должны).
Shift + ↓↑ работает как установка нового одинакового значения элементам, а не инвертирование того, что у них было.
Ошибка 5
Пример с чекбоксами (http://screenshots.ptsecurity.com/mozornin-2020-02-10_20-03-14.png)
Если добавлены чекбокс, то автоселекта не должно быть, только пробелом. По умолчания должен ходить фокус, селект включается явно каждый раз (если можно будет с shift, то отлично).
Ошибка 6
Пример с чекбоксами (http://screenshots.ptsecurity.com/mozornin-2020-02-10_20-03-14.png)
Последний элемент нельзя развыбрать
В этом примере нет мультиселекта, это пример с группами (по параметрах вроде бы видно).
ок, попровлю.
Нужно описать ожидаемое поведение.
ок, поправлю.
быстрее всего включен no-autoselect - поправлю. А что насчет дерева, не смотрел ? В итоге: Нужно описать и зафиксировать требования к мультиселекту, наверно нужно добавить сюда Нужно зафиксировать дефолтные параметры для auto-select, no-unselect и т.д. при включенном мультиселекте (дефолтное оно же checkbox и keyboard). Насколько Я помню мы вроде бы о чем то договаривались, но нигде не зафиксировали. Я в свою очередь поправлю все остальные мелочи в примерах. |
|
Я попрошу Женю, который писал хоткеи, добавить описание сложных случаев 2, 3, 4 в раздел хоткеев. Давай починим здесь все, кроме 2, 3, 4 и вольем. |
|
Preview docs changes for fb9e137 at https://positive-js.github.io/mosaic-previews/pr400-fb9e137/ |
…ift and ctrl (#UIM-156) (#400) * fixed input parameters and focus * fixed selection with shift * fixed selection and focus bugs * fixed tests * in dev example added parameter that switched on 'multiple' without checkboxes * selecting in list and tree * fixed tests * fixed bugs
No description provided.