Skip to content

Add resizing of FLA panels#1068

Merged
1Blademaster merged 2 commits into
mainfrom
994-feature-allow-resizing-of-fla-graph
Mar 7, 2026
Merged

Add resizing of FLA panels#1068
1Blademaster merged 2 commits into
mainfrom
994-feature-allow-resizing-of-fla-graph

Conversation

@1Blademaster
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings March 7, 2026 14:14
@1Blademaster 1Blademaster linked an issue Mar 7, 2026 that may be closed by this pull request
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

Adds resizable panel support to the Falcon Log Analyser (FLA) UI and stores the resulting layout split sizes in Redux so the UI can restore the last-used proportions within the session.

Changes:

  • Added panelLayout state + reducers/selectors for horizontal and vertical panel splits in logAnalyserSlice.
  • Refactored MainDisplay to use react-resizable-panels for horizontal (sidebar/content) and vertical (graph/tabs) resizing, persisting layout via debounced Redux updates.
  • Updated Graph container layout to better fill the available height in the new resizable layout (and adjusted resize-handle styling).

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.

File Description
gcs/src/redux/slices/logAnalyserSlice.js Adds panel layout state, reducers, and selectors to support persisted split sizes.
gcs/src/components/fla/mainDisplay.jsx Replaces fixed columns with resizable panels and dispatches layout updates to Redux.
gcs/src/components/fla/graph.jsx Adjusts graph container sizing to work correctly inside resizable panels; updates handle styling.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread gcs/src/redux/slices/logAnalyserSlice.js Outdated
Comment thread gcs/src/redux/slices/logAnalyserSlice.js Outdated
Comment thread gcs/src/components/fla/mainDisplay.jsx
Comment thread gcs/src/components/fla/mainDisplay.jsx
Comment thread gcs/src/components/fla/graph.jsx
@1Blademaster 1Blademaster merged commit 6edf413 into main Mar 7, 2026
5 checks passed
@1Blademaster 1Blademaster deleted the 994-feature-allow-resizing-of-fla-graph branch March 7, 2026 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] Allow resizing of FLA graph

2 participants