Skip to content

Заготовку можно править на месте и двигать стрелками - #49

Merged
akalikbergenov merged 2 commits into
akalikbergenov:mainfrom
GeorgeSGN:feat/edit-snippets
Aug 24, 2026
Merged

Заготовку можно править на месте и двигать стрелками#49
akalikbergenov merged 2 commits into
akalikbergenov:mainfrom
GeorgeSGN:feat/edit-snippets

Conversation

@GeorgeSGN

Copy link
Copy Markdown

Двойной клик по заготовке разворачивает её в те же два поля, что и форма создания. Enter сохраняет, Esc отменяет, клик мимо сохраняет — уйти от строки, которую только что поправил, это не способ выбросить правку.

Правка заготовки

Стрелки рядом с крестиком задают порядок: список — это приоритет, и то, за чем тянутся чаще всего, должно стоять сверху и там оставаться. Порядок пишется в файл, а не живёт в представлении.

Правка идёт заменой на месте, а не «удалить и добавить»: идентичность заготовки — её имя, поэтому переименование делает её другой записью, и добавление отправило бы строку в начало списка прямо под руками.

Стёртое значение отменяет правку, а не удаляет строку: удаление уже есть у крестика, и терять заготовку из-за случайного ⌘A — плохой размен.

Складывание панели сохраняет начатое — иначе набранное пропадало бы молча, когда курсор ушёл с чёлки.

Первым коммитом идёт #47: правка опирается на идентичность заготовки, и без него переименование в строке с одинаковыми именами вело бы себя странно. Если #47 будет смержен, здесь останется один коммит.

@ispy4you

Copy link
Copy Markdown
Contributor

Заметил, что правка Snippet.id здесь дословно совпадает с #47, а весь этот PR целиком (включая pointerStyle из #48) входит в #50. Не уверен, входило ли это в план — если ветки создавались независимо, а не одна от другой, стоит на это взглянуть, чтобы не разбирать один и тот же диф трижды.

@akalikbergenov
akalikbergenov merged commit 616292e into akalikbergenov:main Aug 24, 2026
1 check passed
Davich02 pushed a commit to Davich02/cyclop that referenced this pull request Aug 24, 2026
Конфликт был один и содержательный: akalikbergenov#44 чинил залипание раскрытой панели
в старом NotchController, а здесь он разобран на общий контроллер и
NotchScreenPanel на каждый дисплей. Порядок «сначала клавиатура, потом
свёртка» тут уже был; чего не было — явного releaseKey и запроса на
перерисовку, потому что acceptsKeyboard перестал отдавать ключ сам.
Оба перенесены в NotchScreenPanel, где теперь живёт закрытие.

Остальное сошлось само: closeDelay 0.05 и свой стиль тумблера из akalikbergenov#42,
заготовки из akalikbergenov#47 и akalikbergenov#49, скрытие из akalikbergenov#51.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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