mirror of
https://github.com/9technologygroup/patchmon.net.git
synced 2025-11-20 22:48:11 +00:00
fix(frontend): elements with button role can be changed to button
This commit is contained in:
@@ -2336,11 +2336,11 @@ const ColumnSettingsModal = ({
|
|||||||
|
|
||||||
<div className="space-y-2">
|
<div className="space-y-2">
|
||||||
{columnConfig.map((column, index) => (
|
{columnConfig.map((column, index) => (
|
||||||
<div
|
<button
|
||||||
key={column.id}
|
key={column.id}
|
||||||
|
type="button"
|
||||||
draggable
|
draggable
|
||||||
tabIndex={0}
|
tabIndex={0}
|
||||||
role="button"
|
|
||||||
aria-label={`Drag to reorder ${column.label} column`}
|
aria-label={`Drag to reorder ${column.label} column`}
|
||||||
onDragStart={(e) => handleDragStart(e, index)}
|
onDragStart={(e) => handleDragStart(e, index)}
|
||||||
onDragOver={handleDragOver}
|
onDragOver={handleDragOver}
|
||||||
@@ -2351,7 +2351,7 @@ const ColumnSettingsModal = ({
|
|||||||
// Focus handling for keyboard users
|
// Focus handling for keyboard users
|
||||||
}
|
}
|
||||||
}}
|
}}
|
||||||
className={`flex items-center justify-between p-3 border rounded-lg cursor-move ${
|
className={`flex items-center justify-between p-3 border rounded-lg cursor-move w-full text-left ${
|
||||||
draggedIndex === index
|
draggedIndex === index
|
||||||
? "opacity-50"
|
? "opacity-50"
|
||||||
: "hover:bg-secondary-50 dark:hover:bg-secondary-700"
|
: "hover:bg-secondary-50 dark:hover:bg-secondary-700"
|
||||||
@@ -2378,7 +2378,7 @@ const ColumnSettingsModal = ({
|
|||||||
<EyeOffIcon className="h-4 w-4" />
|
<EyeOffIcon className="h-4 w-4" />
|
||||||
)}
|
)}
|
||||||
</button>
|
</button>
|
||||||
</div>
|
</button>
|
||||||
))}
|
))}
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user