Skip to content

fix(blocksettings): added a minimum width to block settings#1432

Merged
johbaxter merged 1 commit intodevfrom
1405-block-settings-minimum-size
Jul 18, 2025
Merged

fix(blocksettings): added a minimum width to block settings#1432
johbaxter merged 1 commit intodevfrom
1405-block-settings-minimum-size

Conversation

@Shubham009-prog
Copy link
Copy Markdown
Contributor

Description

Fixed Minimum width of Block Setting Panel

Changes Made

Just changed the minSize in Workspace options

@Shubham009-prog Shubham009-prog self-assigned this Jul 4, 2025
@Shubham009-prog Shubham009-prog requested a review from a team as a code owner July 4, 2025 12:36
@Shubham009-prog Shubham009-prog linked an issue Jul 4, 2025 that may be closed by this pull request
2 tasks
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jul 4, 2025

@CodiumAI-Agent /describe

@QodoAI-Agent
Copy link
Copy Markdown

Title

fix(blocksettings): added a minimum width to block settings


User description

Description

Fixed Minimum width of Block Setting Panel

Changes Made

Just changed the minSize in Workspace options


PR Type

Bug fix


Description

  • Introduce minimum width for block settings panel

  • Add BLOCK_SETTINGS_MIN_WIDTH constant (450)

  • Update right border size to use constant

  • Define minSize for layout stability


Changes diagram

flowchart LR
  A["Default workspace options"] -- "define min width" --> B["BLOCK_SETTINGS_MIN_WIDTH (450)"]
  B -- "apply to right border" --> C["panel size & minSize"]
Loading

Changes walkthrough 📝

Relevant files
Bug fix
BlocksWorkspace.tsx
Enforce minimum width for block settings panel                     

packages/client/src/components/blocks-workspace/BlocksWorkspace.tsx

  • Added BLOCK_SETTINGS_MIN_WIDTH constant with value 450
  • Updated right border size to use new constant
  • Added minSize property to enforce the minimum width
  • +3/-1     

    Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • @github-actions
    Copy link
    Copy Markdown

    github-actions bot commented Jul 4, 2025

    @CodiumAI-Agent /review

    @QodoAI-Agent
    Copy link
    Copy Markdown

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    Constant Configurability

    The new BLOCK_SETTINGS_MIN_WIDTH is a hard-coded value; consider making it configurable or documenting its rationale to ensure consistency with design guidelines.

    const BLOCK_SETTINGS_MIN_WIDTH = 450;
    Responsive Layout

    Setting both size and minSize to 450px for the right border may cause overflow or layout issues on smaller viewports; verify the panel behaves as expected across different screen sizes.

    size: BLOCK_SETTINGS_MIN_WIDTH,
    minSize: BLOCK_SETTINGS_MIN_WIDTH,

    @github-actions
    Copy link
    Copy Markdown

    github-actions bot commented Jul 4, 2025

    @CodiumAI-Agent /improve

    @QodoAI-Agent
    Copy link
    Copy Markdown

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    @johbaxter johbaxter self-requested a review July 18, 2025 16:09
    @johbaxter johbaxter merged commit acf0792 into dev Jul 18, 2025
    4 checks passed
    @johbaxter johbaxter deleted the 1405-block-settings-minimum-size branch July 18, 2025 16:24
    @github-actions
    Copy link
    Copy Markdown

    @CodiumAI-Agent /update_changelog

    @QodoAI-Agent
    Copy link
    Copy Markdown

    Changelog updates: 🔄

    2025-07-18 *

    Fixed

    • Enforce a minimum width for the Block Settings panel to prevent layout issues.

    to commit the new content to the CHANGELOG.md file, please type:
    '/update_changelog --pr_update_changelog.push_changelog_changes=true'

    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.

    Block Settings Minimum Size

    3 participants