From d1f4711db555184402d93d49224b2ca8651dade4 Mon Sep 17 00:00:00 2001
From: Wendy Yuchen Sun <wendyyuchensun@gmail.com>
Date: Mon, 11 May 2026 16:27:40 +0800
Subject: [PATCH] fix: group radio inputs with fieldset

---
 src/components/core/radio-group.js | 10 +++++-----
 src/components/link-input-modal.js |  2 +-
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/components/core/radio-group.js b/src/components/core/radio-group.js
index 0fa5e5f..3662836 100644
--- a/src/components/core/radio-group.js
+++ b/src/components/core/radio-group.js
@@ -2,10 +2,10 @@ import React from 'react';
 import PropTypes from 'prop-types';
 import cn from 'classnames';
 
-const RadioGroup = ({ name, label, options, value, onChange }) => {
+const RadioGroup = ({ name, legend, options, value, onChange }) => {
   return (
-    <div className="flex flex-col gap-2">
-      {label && <span className="text-text-primary text-base">{label}</span>}
+    <fieldset>
+      <legend className="text-text-primary text-base mb-2">{legend}</legend>
       <div className="flex items-center">
         {options.map((option) => (
           <div key={option.value} className="flex flex-1 items-center">
@@ -39,13 +39,13 @@ const RadioGroup = ({ name, label, options, value, onChange }) => {
           </div>
         ))}
       </div>
-    </div>
+    </fieldset>
   );
 };
 
 RadioGroup.propTypes = {
   name: PropTypes.string.isRequired,
-  label: PropTypes.string,
+  legend: PropTypes.string.isRequired,
   options: PropTypes.arrayOf(
     PropTypes.shape({
       value: PropTypes.string.isRequired,
diff --git a/src/components/link-input-modal.js b/src/components/link-input-modal.js
index 53c055c..ff28a42 100644
--- a/src/components/link-input-modal.js
+++ b/src/components/link-input-modal.js
@@ -91,7 +91,7 @@ const LinkInputModal = ({ isOpen, onClose, onConfirm }) => {
         />
         <RadioGroup
           name="link-open-method"
-          label={t('openMethod')}
+          legend={t('openMethod')}
           options={[
             { value: 'new-tab', label: t('newTab') },
             { value: 'current-tab', label: t('currentTab') },