Skip to content

Conversation

@alexandrusoare
Copy link
Contributor

@alexandrusoare alexandrusoare commented Dec 9, 2025

User description

SUMMARY

Security section checkboxes are to close to the secure extra field

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

BEFORE

image

AFTER

image

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

  • Has associated issue:
  • Required feature flags:
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • Migration is atomic, supports rollback & is backwards-compatible
    • Confirm DB migration upgrade and downgrade tested
    • Runtime estimates and downtime expectations provided
  • Introduces new feature or API
  • Removes existing feature or API

CodeAnt-AI Description

Fix spacing between Security options and file-upload fields in Database modal

What Changed

  • Security section input block always uses the no-bottom-margin spacing so checkboxes no longer appear too close to the following extra fields
  • The spacing logic for the impersonate and file-upload blocks was corrected so margins are applied when the engine supports file uploads and when file uploads are enabled
  • When the "Schemas allowed for File upload" field appears, it uses a dedicated container class to avoid collapsed spacing with adjacent controls

Impact

✅ Clearer form spacing in Database modal
✅ Fewer layout overlaps between security checkboxes and extra fields
✅ Consistent spacing when toggling file upload options

💡 Usage Guide

Checking Your Pull Request

Every time you make a pull request, our system automatically looks through it. We check for security issues, mistakes in how you're setting up your infrastructure, and common code problems. We do this to make sure your changes are solid and won't cause any trouble later.

Talking to CodeAnt AI

Got a question or need a hand with something in your pull request? You can easily get in touch with CodeAnt AI right here. Just type the following in a comment on your pull request, and replace "Your question here" with whatever you want to ask:

@codeant-ai ask: Your question here

This lets you have a chat with CodeAnt AI about your pull request, making it easier to understand and improve your code.

Example

@codeant-ai ask: Can you suggest a safer alternative to storing this secret?

Preserve Org Learnings with CodeAnt

You can record team preferences so CodeAnt AI applies them in future reviews. Reply directly to the specific CodeAnt AI suggestion (in the same thread) and replace "Your feedback here" with your input:

@codeant-ai: Your feedback here

This helps CodeAnt AI learn and adapt to your team's coding style and standards.

Example

@codeant-ai: Do not flag unused imports.

Retrigger review

Ask CodeAnt AI to review the PR again, by typing:

@codeant-ai: review

Check Your Repository Health

To analyze the health of your code repository, visit our dashboard at https://app.codeant.ai. This tool helps you identify potential issues and areas for improvement in your codebase, ensuring your repository maintains high standards of code health.

@alexandrusoare alexandrusoare marked this pull request as ready for review December 9, 2025 10:50
@dosubot dosubot bot added the change:frontend Requires changing the frontend label Dec 9, 2025
@alexandrusoare alexandrusoare force-pushed the alexandrusoare/fix/space-between-configs branch from fc84eb7 to db250b9 Compare December 11, 2025 10:19
@codeant-ai-for-open-source
Copy link
Contributor

CodeAnt AI is reviewing your PR.

@codeant-ai-for-open-source codeant-ai-for-open-source bot added the size:XS This PR changes 0-9 lines, ignoring generated files label Dec 11, 2025
@codeant-ai-for-open-source
Copy link
Contributor

Nitpicks 🔍

🔒 No security issues identified
⚡ Recommended areas for review

  • Conditional CSS logic
    Several StyledInputContainer usages changed their conditional css expressions (inverted conditions). Verify the intended spacing rules across combinations of isFileUploadSupportedByEngine and db?.allow_file_upload. Confirm these conditions do not unintentionally collapse margins or leave empty style objects being passed to the css prop.

  • New className added
    A new className="extra-container" was added for the schemas upload block to avoid collapsed spacing. Ensure the CSS for extra-container exists in the corresponding styles (or prefer a styled-prop). If the class is missing this will have no effect.

@codeant-ai-for-open-source
Copy link
Contributor

CodeAnt AI finished reviewing your PR.

@geido geido added the 🎪 ⚡ showtime-trigger-start Create new ephemeral environment for this PR label Dec 11, 2025
@github-actions github-actions bot added 🎪 db250b9 🚦 building Environment db250b9 status: building 🎪 db250b9 📅 2025-12-11T11-33 Environment db250b9 created at 2025-12-11T11-33 🎪 db250b9 ⌛ 48h Environment db250b9 expires after 48h 🎪 db250b9 🤡 geido Environment db250b9 requested by geido and removed 🎪 ⚡ showtime-trigger-start Create new ephemeral environment for this PR labels Dec 11, 2025
@github-actions
Copy link
Contributor

🎪 Showtime is building environment on GHA for db250b9

@github-actions github-actions bot added 🎪 db250b9 🚦 deploying Environment db250b9 status: deploying 🎪 db250b9 🚦 running Environment db250b9 status: running 🎪 🎯 db250b9 Active environment pointer - db250b9 is receiving traffic 🎪 db250b9 🌐 54.202.195.44:8080 Environment db250b9 URL: http://54.202.195.44:8080 (click to visit) and removed 🎪 db250b9 🚦 building Environment db250b9 status: building 🎪 db250b9 🚦 deploying Environment db250b9 status: deploying 🎪 db250b9 🚦 running Environment db250b9 status: running 🎪 🎯 db250b9 Active environment pointer - db250b9 is receiving traffic labels Dec 11, 2025
@github-actions
Copy link
Contributor

🎪 Showtime deployed environment on GHA for db250b9

Environment: http://54.202.195.44:8080 (admin/admin)
Lifetime: 48h auto-cleanup
Updates: New commits create fresh environments automatically

@alexandrusoare alexandrusoare force-pushed the alexandrusoare/fix/space-between-configs branch from 12b4d8f to bc7cd09 Compare December 12, 2025 14:34
@github-actions github-actions bot added 🎪 bc7cd09 🚦 building Environment bc7cd09 status: building 🎪 bc7cd09 📅 2025-12-12T14-35 Environment bc7cd09 created at 2025-12-12T14-35 🎪 bc7cd09 ⌛ 48h Environment bc7cd09 expires after 48h 🎪 bc7cd09 🤡 alexandrusoare Environment bc7cd09 requested by alexandrusoare labels Dec 12, 2025
@github-actions
Copy link
Contributor

🎪 Showtime is building environment on GHA for bc7cd09

@github-actions github-actions bot added 🎪 bc7cd09 🚦 deploying Environment bc7cd09 status: deploying 🎪 bc7cd09 🚦 running Environment bc7cd09 status: running 🎪 🎯 bc7cd09 Active environment pointer - bc7cd09 is receiving traffic 🎪 bc7cd09 🌐 35.87.199.198:8080 Environment bc7cd09 URL: http://35.87.199.198:8080 (click to visit) and removed 🎪 bc7cd09 🚦 building Environment bc7cd09 status: building 🎪 bc7cd09 🚦 deploying Environment bc7cd09 status: deploying 🎪 bc7cd09 🚦 running Environment bc7cd09 status: running 🎪 🎯 bc7cd09 Active environment pointer - bc7cd09 is receiving traffic 🎪 12b4d8f 📅 2025-12-12T10-08 Environment 12b4d8f created at 2025-12-12T10-08 🎪 12b4d8f 🤡 alexandrusoare Environment 12b4d8f requested by alexandrusoare 🎪 12b4d8f 🚦 running Environment 12b4d8f status: running 🎪 12b4d8f ⌛ 48h Environment 12b4d8f expires after 48h 🎪 12b4d8f 🌐 35.95.60.120:8080 Environment 12b4d8f URL: http://35.95.60.120:8080 (click to visit) labels Dec 12, 2025
@github-actions
Copy link
Contributor

🎪 Showtime deployed environment on GHA for bc7cd09

Environment: http://35.87.199.198:8080 (admin/admin)
Lifetime: 48h auto-cleanup
Updates: New commits create fresh environments automatically

@geido
Copy link
Member

geido commented Dec 12, 2025

@alexandrusoare I think the PR introduces some inconsistency with the checkboxes. In this PR they look pretty close to each other. Before, they looked like they had more space, and across different sections, they now also look inconsistent.

Screenshot 2025-12-12 at 20 05 48

@github-actions github-actions bot removed 🎪 bc7cd09 🌐 35.87.199.198:8080 Environment bc7cd09 URL: http://35.87.199.198:8080 (click to visit) 🎪 bc7cd09 📅 2025-12-12T14-35 Environment bc7cd09 created at 2025-12-12T14-35 🎪 bc7cd09 🤡 alexandrusoare Environment bc7cd09 requested by alexandrusoare 🎪 bc7cd09 ⌛ 48h Environment bc7cd09 expires after 48h 🎪 bc7cd09 🚦 running Environment bc7cd09 status: running labels Dec 14, 2025
@codeant-ai-for-open-source
Copy link
Contributor

CodeAnt AI is running Incremental review

@alexandrusoare
Copy link
Contributor Author

@alexandrusoare I think the PR introduces some inconsistency with the checkboxes. In this PR they look pretty close to each other. Before, they looked like they had more space, and across different sections, they now also look inconsistent.

Screenshot 2025-12-12 at 20 05 48

I thought that removing the space looked better, but you are right that it should respect the current pattern, and the current pattern is that there is more space between the checkboxes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

change:frontend Requires changing the frontend size/S size:XS This PR changes 0-9 lines, ignoring generated files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants