Skip to content

Merge develop into master (15 Nov)#2696

Merged
gorkem-bwl merged 22 commits intomasterfrom
develop
Nov 16, 2025
Merged

Merge develop into master (15 Nov)#2696
gorkem-bwl merged 22 commits intomasterfrom
develop

Conversation

@HarshP4585
Copy link
Copy Markdown
Collaborator

Describe your changes

  • Merge develop into master (15 Nov)

Write your issue number after "Fixes "

Enter the corresponding issue number after "Fixes #"

Please ensure all items are checked off before requesting a review:

  • I deployed the code locally.
  • I have performed a self-review of my code.
  • I have included the issue # in the PR.
  • I have labelled the PR correctly.
  • The issue I am working on is assigned to me.
  • I have avoided using hardcoded values to ensure scalability and maintain consistency across the application.
  • I have ensured that font sizes, color choices, and other UI elements are referenced from the theme.
  • My pull request is focused and addresses a single, specific feature.
  • If there are UI changes, I have attached a screenshot or video to this PR.

MuhammadKhalilzadeh and others added 22 commits November 15, 2025 10:33
…utton-disable

Create demo data button commented for now
….develop

Enhancing login alert messages and dashboard spacings
….develop

Backend bug fix for use-case and demo data creation/deletion
Updated NewControlPane to use the StandardModal component for consistent styling and behavior across the application.

Changes:
- Replaced raw Modal component with StandardModal
- Removed ~100 lines of custom modal container styling
- Moved title and description to StandardModal props for gradient header
- Leveraged StandardModal's built-in footer with Cancel/Save buttons
- Removed duplicate close button and save button implementations
- Wrapped content in Stack with spacing={6} for consistent 48px spacing
- Maintained 800px maxWidth for wide content layout
- Properly integrated isSubmitting state with StandardModal

Benefits:
- Consistent modal appearance across the application
- Reduced code duplication and maintenance burden
- Improved accessibility with StandardModal's built-in features
- Better UX with consistent gradient headers and footers

Co-Authored-By: Gorkem Cetin <gorkem.cetin@verifywise.ai>
- Extract flexGrow, minWidth, maxWidth from sx and apply to select-wrapper
- Remove maxWidth constraint from inputStyles to allow fields to expand
- Reduce gaps between form rows and fields for better spacing
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
…ol-modal

Standardize control pane modal with StandardModal component
Changed alignItems from 'center' to 'flex-end' to ensure input fields
align at the bottom across both rows, fixing vertical misalignment.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Changed from flexGrow: 1 to flex: 1 (with flexBasis: 0) to ensure
all fields distribute space equally and have identical widths.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Calculate exact width for 3 fields accounting for gaps between them.
Each field gets (100% - 64px) / 3 width with flexShrink: 0.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Update DatePicker to extract and apply layout props (width, flexGrow,
minWidth, maxWidth) to wrapper Stack instead of MuiDatePicker. This
matches the Select component pattern and ensures all form fields in
the Dropdowns component have equal widths.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
…ol-modal

Fix form field alignment and equal width in control modal
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Nov 16, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch develop

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@gorkem-bwl gorkem-bwl merged commit 63ca344 into master Nov 16, 2025
10 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.

3 participants