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

Исправления и улучшения вкладок с машинами состояний #3

Closed
wants to merge 173 commits into from

Conversation

Roundabout1
Copy link

Готово:

  • модалка с удалением машины состояний больше не вылезает при нажатии на правую кнопку мыши на компонент, вместо этого открывается вкладка с машиной состояний
  • при создании вкладки для машины состояний с пустым названием, для имени вкладки выбирается ID машины состояний, а не ID канваса
  • добавлена функция переименования вкладки
  • добавлена валидация формы создания/редактирования машины состояний: запрещается создавать машину состояний без платформы или с повторяющимся именем

Проблемы:

  • неизвестно, что нужно делать при переключении с платформы "Берлоги" на платформу "Ардуино", так как они используют разные типы компонентов, возможно стоит это запретить. При этом переключение между платформами "Ардуино" или между платформами "Берлога" по-идее должно работать.
  • вкладка с первой машиной состояний имеет название "editor", однако это не совпадает с именем или ID машины состояний, из-за чего возникают проблемы, например при нажатии на правую кнопку мыши открывается новая вкладка, но с другим названием, которое соответствует имени или ID машины, что приводит к крашу IDE, при переключении вкладок. Я думаю стоит просто дать какое-нибудь дефолтное название для вкладки и машины состояний.

bryzZz and others added 30 commits April 1, 2024 03:40
* condition building

* remove unused imports

* operatorAlias to object

* Фикс переходов в шаблонах (kruzhok-team#238)

* fix issues
Co-authored-by: bryzZz <tebloev.nikita@mail.ru>
Co-authored-by: bryzZz <nikita.tebloev@polyus-nt.ru>
…ruzhok-team#260)

* Отображение результата прошивки в отдельной вкладке

* Доработка вывода результата прошивки

* Теперь вкладки не зависят от наличия открытого проекта

---------

Co-authored-by: bryzZz <tebloev.nikita@mail.ru>
- Переименование сущностей, Container на EditorView, MachineController на EditorController, EditorManager на EditorData, Node на Shape
- Рефакторинг отрисовки фигур, выделение абстракций: Drawable, Children, Layer
  - Уменьшение связанности контроллера и Children, путём хранения ссылок на объекты а не айдишников
- Перемещение блоков кода из EditorController в соответствующие контроллеры
- Рефакторинг состояний, разделение на две сущности InitialState и State
- Добавление нескольких начальных состояний
  - Изменения внешнего вида начальных состояний
  - Добавление перехода из начального состояния
  - Удалять начальное состояние при удалении состояния, если некуда переместить
  - Перемещать начальное состояние при удалении состояния, если есть куда переместить
  - Перемещать начальное состояние при линковке состояния, если есть куда переместить
  - Помещение всех действий в историю
- Добавление конечных состояний
- Рефакторинг переходов, добавление возможности отсутствия label или trigger внутри него
- Изменение вида иерархии
- Изменения парсера под новый формат

---------

Co-authored-by: L140-beep <kartoshka0302@mail.ru>
* Добавление кнопок для загрузчика и компилятора.

* Исправление по комментариям.
1) Изменены наименования переменных для открытия модальных окон;
2) Изменён текст ошибок.

* Изменено расположение модальных окон в коде.
Раньше сообщение о новом порте не передавалось из main в renderer. Теперь оно передаётся при вызове метода перезапуска загрузчика.
Также сделан небольшой рефакторинг настроек, чтобы избавиться от повторяющегося кода.

Fixes kruzhok-team#286.
Co-authored-by: bryzZz <tebloev.nikita@mail.ru>
Co-authored-by: Maxim Klypin <70793854+L140-beep@users.noreply.github.com>
L140-beep and others added 28 commits September 18, 2024 09:51
* вкладка на сайдбаре для машин состояний

Иконку возможно придётся поменять, но пока не нашлось ничего лучше

* исправление открытия вкладки через кнопку прошить

* рефакторинг кнопки "добавить машину состояний"

* WIP: добавление/редактирование машин состояний

* отправление формы

* фикс редактирования машины состояний

* управление данными

* использование только id вместо имени

при наличии машин с одинаковыми именами возникала проблема - они обе выделялись при клике

* улучшение внешнего вида

- отступы между полями
- одинаковая длина полей

* удаление машины состояний

* Получение и обработка полного списка платформ

* фиксы после слияния веток
удаление по правому клику препятствовало onCallContextMenu
@Roundabout1
Copy link
Author

не та ветка

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.

5 participants