Merged
Conversation
3-pane dashboard (Info, Rules, Servers) with Tab focus cycling, inline CRUD (add/delete rules, rename, clone, delete group), associated server resolution via ports API, and cross-resource navigation to server detail views.
Add column header row to rules pane with dynamic remote column width. Split bottom row into Servers (50%) | Ports (50%) showing MAC addresses, status, and fixed IPs. Tab cycles 4 panes.
- All columns (rules, servers, ports) now size to actual content - Rules: proper ANSI-aware alignment, dynamic proto/ports/remote widths - Servers: header row, name column sized to content - Ports: server name column from DeviceID, sorted by server then MAC - Rule edit: Enter on rule opens pre-filled form (delete old + create new) - Bottom row aligned with top row (info/servers 35%, rules/ports 65%)
Replace the separate security group list + detail views with a unified 5-pane dashboard: SG selector at top, Info | Rules in the middle, Servers | Ports at the bottom. Selecting a different SG instantly updates all detail panes. Context-sensitive action bar: SG CRUD on selector pane, rule CRUD+edit on rules pane, server navigation on servers pane. Esc from server detail returns to SG view via breadcrumb. Removes the sgdetail package entirely — all logic merged into secgroupview.
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
1 participant
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.
Summary
sgdetailpackage — all logic merged intosecgroupviewTest plan
rrenames,cclones from selector/info/ports panes