mirror of
https://github.com/AutoMaker-Org/automaker.git
synced 2026-02-04 09:13:08 +00:00
fix: scrolling issues in phase model selector
- scrolling was broken when we used component inside modal / dialog
This commit is contained in:
@@ -564,10 +564,18 @@ export function PhaseModelSelector({
|
|||||||
|
|
||||||
// The popover content (shared between both modes)
|
// The popover content (shared between both modes)
|
||||||
const popoverContent = (
|
const popoverContent = (
|
||||||
<PopoverContent className="w-[320px] p-0" align={align}>
|
<PopoverContent
|
||||||
|
className="w-[320px] p-0"
|
||||||
|
align={align}
|
||||||
|
onWheel={(e) => e.stopPropagation()}
|
||||||
|
onPointerDownOutside={(e) => e.preventDefault()}
|
||||||
|
>
|
||||||
<Command>
|
<Command>
|
||||||
<CommandInput placeholder="Search models..." />
|
<CommandInput placeholder="Search models..." />
|
||||||
<CommandList ref={commandListRef} className="max-h-[300px]">
|
<CommandList
|
||||||
|
ref={commandListRef}
|
||||||
|
className="max-h-[300px] overflow-y-auto overscroll-contain"
|
||||||
|
>
|
||||||
<CommandEmpty>No model found.</CommandEmpty>
|
<CommandEmpty>No model found.</CommandEmpty>
|
||||||
|
|
||||||
{favorites.length > 0 && (
|
{favorites.length > 0 && (
|
||||||
@@ -626,7 +634,7 @@ export function PhaseModelSelector({
|
|||||||
// Compact mode - just the popover with compact trigger
|
// Compact mode - just the popover with compact trigger
|
||||||
if (compact) {
|
if (compact) {
|
||||||
return (
|
return (
|
||||||
<Popover open={open} onOpenChange={setOpen}>
|
<Popover open={open} onOpenChange={setOpen} modal={false}>
|
||||||
<PopoverTrigger asChild>{compactTrigger}</PopoverTrigger>
|
<PopoverTrigger asChild>{compactTrigger}</PopoverTrigger>
|
||||||
{popoverContent}
|
{popoverContent}
|
||||||
</Popover>
|
</Popover>
|
||||||
@@ -649,7 +657,7 @@ export function PhaseModelSelector({
|
|||||||
</div>
|
</div>
|
||||||
|
|
||||||
{/* Model Selection Popover */}
|
{/* Model Selection Popover */}
|
||||||
<Popover open={open} onOpenChange={setOpen}>
|
<Popover open={open} onOpenChange={setOpen} modal={false}>
|
||||||
<PopoverTrigger asChild>{fullTrigger}</PopoverTrigger>
|
<PopoverTrigger asChild>{fullTrigger}</PopoverTrigger>
|
||||||
{popoverContent}
|
{popoverContent}
|
||||||
</Popover>
|
</Popover>
|
||||||
|
|||||||
Reference in New Issue
Block a user