Skip to content

Modernize connection management dialogs - #103

Merged
bifrost0x merged 2 commits into
mainfrom
dev/management-dialog-ui-consistency
Aug 13, 2026
Merged

Modernize connection management dialogs#103
bifrost0x merged 2 commits into
mainfrom
dev/management-dialog-ui-consistency

Conversation

@bifrost0x

Copy link
Copy Markdown
Owner

Summary

  • align the saved connection editor with the Quick Connect card hierarchy
  • keep profile actions visible while only the form content scrolls
  • modernize Jump Hosts, SSH Keys, and Settings with responsive manager cards
  • replace modal close spans with keyboard-focusable buttons across workspace, SFTP, and admin dialogs
  • add focused desktop, mobile, scrolling, and accessibility browser coverage

Validation

  • 4 focused Playwright UI tests
  • 169 JavaScript unit tests
  • 38 targeted template contract tests
  • JavaScript lint
  • vendored asset integrity check
  • Docker image build and visual verification on the local preview

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

Copy link
Copy Markdown

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: b67c47ae7d

ℹ️ 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 templates/index.html
Comment thread static/css/style.css
@bifrost0x
bifrost0x merged commit b794b14 into main Aug 13, 2026
15 of 16 checks passed
@bifrost0x
bifrost0x deleted the dev/management-dialog-ui-consistency branch August 13, 2026 12:00
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.

1 participant