Compare commits

...

2 Commits

Author SHA1 Message Date
openhands
cb10d6e55b Fix clickable area issue in API key modal buttons
- Replace React Fragment wrappers with proper div containers in modal footers
- Affects create-api-key-modal, new-api-key-modal, and delete-api-key-modal
- Ensures buttons are only clickable within their actual visual boundaries
- Maintains existing styling and functionality
2025-07-21 14:32:08 +00:00
openhands
b09fa7951e Fix clickable area issue in Settings/Integrations buttons
- Wrap buttons in a container div to prevent full-width clickable area
- Buttons now only respond to clicks on the actual button content
- Fixes issue where clicking anywhere in the button row would trigger button actions
2025-07-21 14:24:51 +00:00
4 changed files with 8 additions and 8 deletions

View File

@@ -50,7 +50,7 @@ export function CreateApiKeyModal({
};
const modalFooter = (
<>
<div className="flex gap-2">
<BrandButton
type="button"
variant="primary"
@@ -73,7 +73,7 @@ export function CreateApiKeyModal({
>
{t(I18nKey.BUTTON$CANCEL)}
</BrandButton>
</>
</div>
);
return (

View File

@@ -40,7 +40,7 @@ export function DeleteApiKeyModal({
if (!keyToDelete) return null;
const modalFooter = (
<>
<div className="flex gap-2">
<BrandButton
type="button"
variant="danger"
@@ -63,7 +63,7 @@ export function DeleteApiKeyModal({
>
{t(I18nKey.BUTTON$CANCEL)}
</BrandButton>
</>
</div>
);
return (

View File

@@ -29,7 +29,7 @@ export function NewApiKeyModal({
if (!newlyCreatedKey) return null;
const modalFooter = (
<>
<div className="flex gap-2">
<BrandButton
type="button"
variant="primary"
@@ -40,7 +40,7 @@ export function NewApiKeyModal({
<BrandButton type="button" variant="secondary" onClick={onClose}>
{t(I18nKey.BUTTON$CLOSE)}
</BrandButton>
</>
</div>
);
return (

View File

@@ -175,7 +175,7 @@ function GitSettingsScreen() {
<div className="flex gap-6 p-6 justify-end border-t border-t-tertiary">
{!shouldRenderExternalConfigureButtons && (
<>
<div className="flex gap-6">
<BrandButton
testId="disconnect-tokens-button"
name="disconnect-tokens-button"
@@ -196,7 +196,7 @@ function GitSettingsScreen() {
{!isPending && t("SETTINGS$SAVE_CHANGES")}
{isPending && t("SETTINGS$SAVING")}
</BrandButton>
</>
</div>
)}
</div>
</form>