Conjoint-ly / vue-bootstrap-editable-table

A Vue Bootstrap component for editable tables
The Unlicense
0 stars 7 forks source link

Кнопки для перетаскивания колонок и строк #27

Closed jumpingrabbit closed 3 years ago

jumpingrabbit commented 3 years ago
jumpingrabbit commented 3 years ago

У меня не получается сделать перетаскивание по кнопке, но до сих пор работает перетаскивание при нажатии на само название колонки:

cannot drag and drop


Сразу скажу, что меня смущает требование последней задачи #27, для неё я бы всё таки предложил другой вариант реализации/визуализации.

@sambassistА какая ещё возможна реализация?

ghost commented 3 years ago

@jumpingrabbit, на мой взгляд было бы логичнее оставить перетаскивание так, как оно было, как везде и делается, просто при наведении на шапку этой колонки иконку типа колонки заменять на иконку перетаскивания, слева, перед названием.

Подобным образом можно поступить и с кнопкой удаления, отображать её справа, при наведении на колонку, либо примерно так, как это было раньше, в правом верхнем углу, но дополнительно сделать алерт или модальное окно с вопросом о том, уверен ли пользователь, что он хочет удалить эту колонку. Аналогично со строками.

ghost commented 3 years ago

У меня не получается сделать перетаскивание по кнопке, но до сих пор работает перетаскивание при нажатии на само название колонки:

cannot drag and drop

Сразу скажу, что меня смущает требование последней задачи #27, для неё я бы всё таки предложил другой вариант реализации/визуализации.

@jumpingrabbit, перетаскивание за кнопку будет работать в случае, если кнопку направить на область самой шпки с колонками.

Данную реализацию можно извернуться и довести до конца, но я решил озвучить своё мнение, потмучто иначе начинает напоминать изобретение велосипеда без особой на то необходимости.

jumpingrabbit commented 3 years ago

@jumpingrabbit, перетаскивание за кнопку будет работать в случае, если кнопку направить на область самой шпки с колонками.

@sambassist , у меня не получается так сделать.

Можно пожалуйста сделать, чтобы можно было кнопку использовать для перетаскивания.

ghost commented 3 years ago

@jumpingrabbit, перетаскивание за кнопку будет работать в случае, если кнопку направить на область самой шпки с колонками.

@sambassist , у меня не получается так сделать.

Можно пожалуйста сделать, чтобы можно было кнопку использовать для перетаскивания.

Хорошо

ghost commented 3 years ago

@jumpingrabbit, Создал дополнительный реквест

jumpingrabbit commented 3 years ago

Что понимается под "Кнопки для перетаскивания колонок и строк". Если колонки и строки уже переносятся.

image

ghost commented 3 years ago

Доброго времени суток.

И к чему это?

чт, 4 мар. 2021 г., 05:57 Nik Samoylov notifications@github.com:

Что понимается под "Кнопки для перетаскивания колонок и строк". Если колонки и строки уже переносятся.

[image: image] https://user-images.githubusercontent.com/13073856/109888845-61c4ce00-7cd8-11eb-8ed2-5919c852cecc.png

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Conjoint-ly/vue-bootstrap-editable-table/issues/27#issuecomment-790163171, or unsubscribe https://github.com/notifications/unsubscribe-auth/AD6JLVOPOKS6Z3PITWD6YE3TB3EHFANCNFSM4YBYEHRA .

jumpingrabbit commented 3 years ago

Доброго времени суток. И к чему это? чт, 4 мар. 2021 г., 05:57 Nik Samoylov notifications@github.com: Что понимается под "Кнопки для перетаскивания колонок и строк". Если колонки и строки уже переносятся. [image: image] https://user-images.githubusercontent.com/13073856/109888845-61c4ce00-7cd8-11eb-8ed2-5919c852cecc.png — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#27 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AD6JLVOPOKS6Z3PITWD6YE3TB3EHFANCNFSM4YBYEHRA .

@sambassist Это уже для другого программиста.

DwarGrak commented 3 years ago

У меня нет никаких кнопок для удаления/перемещения колонок. Точно версия репозитория актуальна?

jumpingrabbit commented 3 years ago

думаю, что не очень актуальная. Надо смержить мастер.

image

ghost commented 3 years ago

Я все вижу)

вт, 30 мар. 2021 г., 07:41 Nik Samoylov @.***>:

думаю, что не очень актуальная. Надо смержить мастер.

[image: image] https://user-images.githubusercontent.com/13073856/112920669-25f51b00-9155-11eb-8f27-36cec8ded435.png

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Conjoint-ly/vue-bootstrap-editable-table/issues/27#issuecomment-809840781, or unsubscribe https://github.com/notifications/unsubscribe-auth/AD6JLVKIMO6NKJYY4PM6W63TGET25ANCNFSM4YBYEHRA .

jumpingrabbit commented 3 years ago

Я все вижу)

Очень хорошо )

DwarGrak commented 3 years ago

Готово на dev

jumpingrabbit commented 3 years ago
DwarGrak commented 3 years ago

image Что не так с последним пунктом? Показывается ячейка, которую переносим

jumpingrabbit commented 3 years ago

Всё вижу - теперь работает.