Skip to content

Conversation

@ricardoantoniocm
Copy link
Contributor

This pull request makes small UI and styling adjustments to the Create Project modal in the Label Studio web app. The changes focus on improving the appearance of the modal header and refining button behavior.

Screenshots

Before

image

After

image

UI and Styling Improvements:

  • Increased the font size of the modal header title from 20px to 24px and updated the selector from .modal__header to .modal-ls__header in CreateProject.scss for better styling consistency.
  • Added a minimum width of 224px to elements with the .space-ls_direction_horizontal class inside the modal header to improve layout alignment.

Button Behavior Adjustments:

  • Removed the size="small" prop from both the Cancel and Save buttons in CreateProject.jsx to standardize button sizing. [1] [2]
  • Added the waitingClickable={false} prop to the Save button to prevent user interaction while the button is in a waiting state.

@ricardoantoniocm ricardoantoniocm self-assigned this Oct 27, 2025
@ricardoantoniocm ricardoantoniocm marked this pull request as ready for review October 27, 2025 20:49
@github-actions github-actions bot added the fix label Oct 27, 2025
@ricardoantoniocm ricardoantoniocm requested review from a team and nass600 October 27, 2025 20:49
@netlify
Copy link

netlify bot commented Oct 27, 2025

Deploy Preview for heartex-docs canceled.

Name Link
🔨 Latest commit 753bde1
🔍 Latest deploy log https://app.netlify.com/projects/heartex-docs/deploys/68fff28c782d720008d88156

@netlify
Copy link

netlify bot commented Oct 27, 2025

Deploy Preview for label-studio-docs-new-theme canceled.

Name Link
🔨 Latest commit 753bde1
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-docs-new-theme/deploys/68fff28c1a9a0c00080f05b1

@netlify
Copy link

netlify bot commented Oct 27, 2025

Deploy Preview for label-studio-playground ready!

Name Link
🔨 Latest commit 753bde1
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-playground/deploys/68fff28cfa2fab0008830ca6
😎 Deploy Preview https://deploy-preview-8723--label-studio-playground.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Oct 27, 2025

Deploy Preview for label-studio-storybook ready!

Name Link
🔨 Latest commit 753bde1
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-storybook/deploys/68fff28c6515b50008ccd817
😎 Deploy Preview https://deploy-preview-8723--label-studio-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@codecov
Copy link

codecov bot commented Oct 27, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.93%. Comparing base (03196ef) to head (753bde1).
⚠️ Report is 3 commits behind head on develop.

❗ There is a different number of reports uploaded between BASE (03196ef) and HEAD (753bde1). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (03196ef) HEAD (753bde1)
pytests 1 0
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #8723      +/-   ##
===========================================
- Coverage    67.03%   59.93%   -7.10%     
===========================================
  Files          790      551     -239     
  Lines        60680    38996   -21684     
  Branches     10311    10311              
===========================================
- Hits         40678    23373   -17305     
+ Misses       19999    15620    -4379     
  Partials         3        3              
Flag Coverage Δ
lsf-e2e 54.22% <ø> (-0.01%) ⬇️
lsf-integration 50.66% <ø> (ø)
lsf-unit 8.34% <ø> (ø)
pytests ?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ricardoantoniocm ricardoantoniocm changed the title fix: FIT-861: LS-DEVELOP: Project Name tab at Create Project modal lo… … fix: FIT-861: LS-DEVELOP: Project Name tab at Create Project modal looks broken Oct 27, 2025
@ricardoantoniocm
Copy link
Contributor Author

ricardoantoniocm commented Oct 27, 2025

/git merge

Workflow run
Successfully merged: create mode 100644 web/libs/ui/src/assets/icons/clipboard-check.svg

@robot-ci-heartex robot-ci-heartex merged commit 60a91b8 into develop Oct 28, 2025
45 of 46 checks passed
@robot-ci-heartex robot-ci-heartex deleted the fb-fit-861 branch October 28, 2025 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants