ui: чертежи не уезжают под редактор объекта + честный тогл кнопки#9
Merged
Conversation
1. BlueprintPanel сдвигается левее (right:334px) когда открыт ConfigPanel (правый drawer 320px) — обе панели видны, не накладываются; transition синхронно со слайдом редактора (0.2s). 2. Убрана плашка-реоткрывашка «⬓ BLUEPRINTS ▲»: закрыто = панель полностью скрыта. Кнопка «Чертежи» в TopBar (и клавиша B) стала честным тоглом показать/скрыть, а не сворачивать в дубль-плашку в том же углу. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
PR делает два UX-фикса панели чертежей (BlueprintPanel) в web/: (1) сдвигает панель влево при открытом правом ConfigPanel, чтобы они не перекрывались; (2) убирает “плашку-реоткрывашку” и делает видимость панели честным show/hide по TopBar-кнопке/клавише B.
Changes:
- Добавлено определение “ConfigPanel открыт” и CSS-смещение
BlueprintPanel(анимацияrightсинхронно 0.2s). - Убрана логика/стили “reopen” кнопки: при закрытии панель полностью скрыта.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| web/src/ui/BlueprintPanel.tsx | Убрана “reopen” плашка, добавлена логика сдвига панели при открытом ConfigPanel, обновлена видимость панели. |
| web/src/ui/BlueprintPanel.css | Добавлен transition по right и правило смещения панели влево при открытом ConfigPanel; удалены стили “reopen” кнопки. |
Comments suppressed due to low confidence (1)
web/src/ui/BlueprintPanel.tsx:79
- Tooltip у кнопки закрытия всё ещё говорит "Collapse dock", хотя плашка-реоткрывашка удалена и теперь панель именно скрывается. Лучше обновить title, чтобы соответствовать новому поведению (и не вводить в заблуждение), заодно можно убрать лишний пробел в className при
configOpen === false.
<div className={`blueprint-panel ${configOpen ? 'config-open' : ''}`}>
<div className="blueprint-header">
<h3 className="blueprint-title">Чертежи</h3>
<button className="blueprint-icon-btn" onClick={handleClose} title="Collapse dock (B)">
▾
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+16
to
+20
| // Редактор объекта (ConfigPanel) — тот же правый край, что и мы. Когда он открыт | ||
| // (выбрана существующая сущность), сдвигаемся левее, чтобы не уезжать под него. | ||
| const selectedEntityId = useStore((state) => state.selectedEntityId); | ||
| const entities = useStore((state) => state.entities); | ||
| const configOpen = !!(selectedEntityId && entities[selectedEntityId]); |
Comment on lines
+19
to
22
| чтобы не оказаться под ним. */ | ||
| .blueprint-panel.config-open { | ||
| right: 334px; /* 320px ширина drawer + 14px зазор */ | ||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #6
Что
Два UX-фикса панели чертежей (BlueprintPanel):
Наложение с редактором объекта. ConfigPanel — правый drawer 320px, BlueprintPanel тоже у правого края → чертежи уезжали под редактор. Теперь при открытом редакторе панель чертежей сдвигается левее (
right: 334px= 320px drawer + 14px зазор), обе видны одновременно;transition: right 0.2sсинхронно со слайдом редактора.Кнопка «Чертежи» не закрывала панель. Раньше «закрытие» сворачивало панель в плашку-реоткрывашку
⬓ BLUEPRINTS ▲в том же углу — визуально казалось, что кнопка ничего не делает, плюс дублировала себя. Убрал плашку: закрыто = панель полностью скрыта. Кнопка «Чертежи» в TopBar (и клавишаB) — честный тогл показать/скрыть, с индикациейactiveна кнопке.Проверка
pnpm typecheck✓,pnpm build✓Замечание
У панели «Результаты» та же схема плашки-реоткрывашки (
⬓ RESULTS ▲) — здесь намеренно не трогал (авто-открытие при результате ракеты — кульминация демо).🤖 Generated with Claude Code