Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: swap the 2 highlighted participants with the space bar #431

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sportog
Copy link
Contributor

@sportog sportog commented Feb 17, 2024

Возможность поменять двух выделенных участников местами с помощью кнопки "пробел".

@daxartio
Copy link
Collaborator

daxartio commented Mar 9, 2024

Я бы предложил пока не использовать space как hotkey для этого. Предлагаю добавить как пункт меню, дальше посмотрим как улучшить работу.

@sergeikobelev
Copy link
Member

Я бы предложил пока не использовать space как hotkey для этого. Предлагаю добавить как пункт меню, дальше посмотрим как улучшить работу.

В принципе этот функционал достаточно безопасный, даже если кто-то случайно нажмет пробел при 2 выделенных участниках, это просто поменяет порядок строк, сами записи не изменятся. Функционал нужен для финишного допиливания стартовой жеребьевки, например, когда хотят развести спортсменов из 2 групп, стартующих подряд на границе стартовых забегов.

В Winorient работает немного по-другому - нужно зажимать пробел при переходе с одной строки на другую и тогда они поменяются местами. Но там нет множественного выделения.

Пункт меню наверное нужен, неизвестно, когда дойдут руки сделать документацию, а так хотя бы по меню пользователи смогут узнать о существовании функционала.

@@ -69,5 +78,7 @@ def entry_single_clicked(index):
self.person_table.clicked.connect(entry_single_clicked)
self.entry_layout.addWidget(self.person_table)

self.person_table.keyPressEvent = self.keyPressEvent
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

После этого станет недоступно редактирование записей через Enter.

@daxartio
Copy link
Collaborator

линтеры также не проходят
нужно запустить авто форматирование кода

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants