fix: Configure test variables form (UI tweaks) #11

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Open

agx-ui wants to merge 1 commit into openai:main from agx-ui:fix/configure-test-variables-form

agx-ui commented Jul 11, 2025

Fixed issues :

Clipped URL input field: The URL input appeared clipped due to insufficient spacing between its container and the element above, causing the top border to be partially hidden.

url

Incorrect label linkage: The "User info" label was incorrectly linked to the requires-login switch via htmlFor="requires-login", causing the switch to toggle when the label was clicked.
Improper address field disabling: The address input was being disabled based on the requiresLogin flag, even though there's no logical connection between them. This incorrectly prevented editing when requiresLogin was false.

Screenshot 2025-07-11 104044

Missing field disabling on submit: The Name and Email fields were not disabled during form submission, allowing changes mid-submit.

agx-ui force-pushed the fix/configure-test-variables-form branch from bde39cd to deae558 Compare

July 11, 2025 11:39


          fix: Configure test variables form (UI tweaks)

9177f67

agx-ui force-pushed the fix/configure-test-variables-form branch from deae558 to 9177f67 Compare

July 11, 2025 11:43

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

Labels

None yet