Skip to content

Lite: misc#13188

Merged
OliverJAsh merged 36 commits into
masterfrom
lite-misc
Apr 5, 2026
Merged

Lite: misc#13188
OliverJAsh merged 36 commits into
masterfrom
lite-misc

Conversation

@OliverJAsh
Copy link
Copy Markdown
Contributor

No description provided.

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
gitbutler-web Skipped Skipped Apr 5, 2026 5:23pm

Request Review

Copilot AI review requested due to automatic review settings April 5, 2026 17:23
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Refactors the Lite workspace UI selection/navigation model and updates related UI/shortcut behavior, while renaming shared “item” styling concepts to “row”.

Changes:

  • Reworks workspace selection to separate “selected item” from “selected file/hunk” and introduces a dedicated Change item type.
  • Replaces the old navigation model with a workspace outline + navigation index (-WorkspaceModel.ts) and updates keyboard shortcut navigation accordingly.
  • Renames shared UI styles from item* to row* across workspace/branches and updates preview close behavior (Escape now maps to ClosePreview).

Reviewed changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
apps/lite/ui/src/routes/project/$id/workspace/route.tsx Adopts new selection model (item vs file), introduces “Change” preview path, and converts UI structure to new row styling.
apps/lite/ui/src/routes/project/$id/workspace/route.module.css Updates padding/line-height vars to --row-* and tweaks layout spacing.
apps/lite/ui/src/routes/project/$id/workspace/-WorkspaceShortcuts.ts Updates shortcut scopes/actions for new item model and adds ClosePreview binding behavior.
apps/lite/ui/src/routes/project/$id/workspace/-WorkspaceModel.ts New outline + navigation index builder for adjacent item/section navigation.
apps/lite/ui/src/routes/project/$id/workspace/-Selection.ts Removes the old navigation/selection model implementation.
apps/lite/ui/src/routes/project/$id/workspace/-OperationSubjects.tsx Renames Changes operation subjects to “ChangesSection*” to match the new conceptual model.
apps/lite/ui/src/routes/project/$id/workspace/-Item.ts Updates item types (splits Changes into Changes + Change) and adds explicit modes for commit/segment.
apps/lite/ui/src/routes/project/$id/workspace/-Editing.ts Removes standalone editing state types in favor of mode-based selection.
apps/lite/ui/src/routes/project/$id/branches/route.tsx Migrates branch rows/actions to the new shared row styling classes.
apps/lite/ui/src/routes/project/$id/branches/route.module.css Updates padding/line-height vars to --row-*.
apps/lite/ui/src/routes/project/$id/-state/selection.ts Adds file to selection state, new normalize helpers, and updates selection resolution API.
apps/lite/ui/src/routes/project/$id/-state/project.ts Ensures SelectFile focuses primary and updates workspace selection consistently.
apps/lite/ui/src/routes/project/$id/-state/layout.ts Introduces ClosePreview action to close fullscreen preview or collapse the preview panel.
apps/lite/ui/src/routes/project/$id/-shared.tsx Updates empty-state class name to rowEmpty.
apps/lite/ui/src/routes/project/$id/-shared.module.css Renames shared styling from item-* to row-* and adjusts hover/action visibility logic.
apps/lite/ui/src/routes/project/$id/-ProjectPreviewLayout.tsx Switches preview close shortcut/button wiring to closePreviewBinding and dispatches ClosePreview.
apps/lite/ui/src/global.css Renames CSS variables from --item-* to --row-* and adds --color-row-hover-bg.

Comment thread apps/lite/ui/src/routes/project/$id/-shared.module.css
Copy link
Copy Markdown
Contributor

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 01c1dccdcc

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread apps/lite/ui/src/routes/project/$id/workspace/route.tsx
Comment thread apps/lite/ui/src/routes/project/$id/workspace/route.tsx
@OliverJAsh OliverJAsh merged commit 258a056 into master Apr 5, 2026
41 checks passed
@OliverJAsh OliverJAsh deleted the lite-misc branch April 5, 2026 19:09
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