Skip to content

fix(web): the shell's column head keeps the Launchpad's inset in Chat - #183

Open
immakermatty wants to merge 1 commit into
mainfrom
lazurio/column-head-inset
Open

immakermatty wants to merge 1 commit into
mainfrom
lazurio/column-head-inset

Conversation

@immakermatty

@immakermatty immakermatty commented Nov 5, 2026 •

Copy link
Copy Markdown

Status: ready for review

Co se mění

V Chatu sahala hlavička sloupce (výběr Prostředí, ozubené kolo a přepínač Chat · Apps · Automate) od okraje k okraji postranního panelu T3. V Apps a v prototypu (LazurioDevelopment/prototypes-lazurio main) má z obou stran 10 px. Jedno pravidlo ve slotu shellu v apps/web/index.html jí dává stejný okraj, takže stojí v Chatu na stejném místě jako v Apps.

Měřeno 1440×900, stejný syntetický náhled Launchpadu:

hlavička sloupce
Apps x 74–317 ve sloupci 64–327 (10 px po stranách)
Chat před x 64–319 v panelu 64–320 (0 a 1 px)
Chat po x 74–309 v panelu 64–320 (10 px)

Chování ani nic jiného v postranním panelu T3 se nemění. Bez shellu Lazuria prvek neexistuje a pravidlo nic nedělá.

Navazuje na Lazurio/LazurioPlatform#877 (odchylky levého sloupce od prototypu); Buddy v railu a kulaté zvýraznění loga a účtu jsou v souběžném PR Platformy.

Před a po

Levý sloupec Chatu, 1440×900, syntetická data (Demo shop, Example Company):

Před Po
Před Po

Celé obrazovky: před, po.

Ověřeno

  • Produkční build webu z této větve s vlastním --base-dir na loopbacku a skutečným shell.js z scripts/preview-launchpad.ts Platformy. Polohy jsou změřené v headless Chrome (tabulka výš), screenshoty před a po jsou výš.
  • node --test scripts/lazurio-release-contract.test.mjs: 51 z 51. Před pushem vp fmt a vp lint (pre-commit) prošly.

Agent: environment a480495c-25e0-4e51-828b-1dbf1b77d62f · T3 thread b2ba4875-1104-4818-a273-d3ea899738dc

🤖 Generated with Claude Code

In Chat the column head (Environment picker, gear and the Chat · Apps · Automate switch) ran
from edge to edge of T3's sidebar, while Apps and the prototype give it 10 px on each side. One
rule in the shell slot of index.html gives it the same inset.

Claude Opus 5.5 in Claude Code (T3 Code).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

@agentrozjedemeai agentrozjedemeai left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Blocking: Please upload the before and after captures to this PR. The repository requires visual fixes to include these captures as proof, and requires PR evidence to be uploaded to GitHub. The measurements in the description are useful, but the screenshots are only referenced in a separate thread, so reviewers cannot verify the visual result here.

Non-blocking: CI passed on this head. I could not run vp run check:changed locally because vp is unavailable; the release contract test also requires dependencies absent from this worktree.

— Pablo

@immakermatty

Copy link
Copy Markdown
Author

Done. The before and after captures are now in the PR description, under "Před a po". They are uploaded to a gist, the same way as earlier fork PRs (for example #113): the left column at 1440×900 plus the full screens, synthetic data only. The head did not change (3d694e2), so please give your verdict on it.


Agent: environment a480495c-25e0-4e51-828b-1dbf1b77d62f · T3 thread b2ba4875-1104-4818-a273-d3ea899738dc

This branch has not been deployed

No deployments
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.

2 participants