Skip to content

Redesign Amounts section with Composition Basis dropdown and phase su…#101

Open
shubham148-st wants to merge 1 commit into
FOSSEE:masterfrom
shubham148-st:basicfix
Open

Redesign Amounts section with Composition Basis dropdown and phase su…#101
shubham148-st wants to merge 1 commit into
FOSSEE:masterfrom
shubham148-st:basicfix

Conversation

@shubham148-st

Copy link
Copy Markdown
Contributor

This PR redesigns the Amounts section within the MaterialStream Results tab to significantly improve usability and data scannability. It replaces the old, deeply nested layout with a clean, top-level tab split, a dedicated Composition Basis dropdown, and specialized phase sub-tabs.

Key Changes

1. Tab Layout Restructuring

  • Replaced the single QToolBox with two distinct, top-level tabs: Amounts and Phase Properties.
  • Added a Composition Basis dropdown at the top, allowing users to toggle between Mole Fraction, Mass Fraction, Mole Flow, and Mass Flow.
    • Positioned Mixture, Liquid, and Vapour sub-tabs directly below the dropdown. Each sub-tab now displays a clean view of the chemical compounds alongside their respective values strictly for the selected basis.
  • Phase Properties Maintenance: The Phase Properties tab completely retains its existing Mixture / Liquid / Vapour accordion layout, ensuring backward compatibility.

2. UI Codebase Cleanup

  • Widget Migration (DockWidgetMaterialStream.ui): Replaced the legacy QToolBox layout in the UI designer file with the updated tabbed configuration.
  • Data Mapping Refactor (DockWidgetMaterialStream.py): Updated the widget setup routines and results population pipeline. Added dynamic switching logic to re-populate compound tables instantly when the selected composition basis changes.
  • Deduplication: Safely eliminated all stale references to deleted tree widgets (mTreeWidget, lTreeWidget, vTreeWidget).
    ameters normally.

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