Skip to content

Conversation

@LaSi5002
Copy link
Contributor

@LaSi5002 LaSi5002 commented Oct 9, 2025

No description provided.

@c-p-schmidt c-p-schmidt requested a review from Copilot October 10, 2025 11:51
Copy link

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

This PR adds functionality to add and delete sections in the web viewer interface. It introduces UI controls for managing sections dynamically, including an autocomplete field for adding new sections and a delete button for removing existing ones.

  • Adds a new top row layout with edit mode switch and section addition controls
  • Implements backend handlers for adding and deleting sections with proper state management
  • Updates the section dropdown visibility logic to handle cases where main and selected sections differ

Reviewed Changes

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

File Description
src/fourc_webviewer/gui_utils.py Adds _top_row function with add section controls, moves edit mode switch, adds delete section button, and updates dropdown visibility logic
src/fourc_webviewer/fourc_webserver.py Implements click_delete_section_button and change_add_section handlers with state initialization for the add section feature

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Collaborator

@dragos-ana dragos-ana left a comment

Choose a reason for hiding this comment

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

Nice work! I added some suggestions, feel free to ask if something is unclear

@dragos-ana dragos-ana self-requested a review October 10, 2025 18:19
@LaSi5002 LaSi5002 force-pushed the feature/add-delete-sections branch from b0f7dba to 60b6483 Compare October 29, 2025 20:46
@LaSi5002 LaSi5002 force-pushed the feature/add-delete-sections branch from bf97e4f to deaf3c9 Compare October 29, 2025 22:57
@dragos-ana dragos-ana merged commit 0503760 into 4C-multiphysics:main Nov 1, 2025
2 checks passed
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