fix(agent): improve model selector legibility
This commit is contained in:
@@ -257,7 +257,7 @@ export function AssistantPanel({ configured, userId, assistantName, assistantMod
|
|||||||
value={selectedModel}
|
value={selectedModel}
|
||||||
onChange={(event) => changeModel(event.target.value)}
|
onChange={(event) => changeModel(event.target.value)}
|
||||||
disabled={modelsLoading || savingModel || isPending}
|
disabled={modelsLoading || savingModel || isPending}
|
||||||
className="h-7 w-full max-w-36 appearance-none truncate rounded-[min(var(--radius-md),10px)] border border-input bg-transparent py-1 pr-7 pl-2.5 text-sm outline-none transition-colors focus-visible:border-ring focus-visible:ring-3 focus-visible:ring-ring/50 disabled:cursor-not-allowed disabled:opacity-50"
|
className="h-9 w-full max-w-36 appearance-none truncate rounded-[min(var(--radius-md),10px)] border border-input bg-[var(--card)] py-0 pr-8 pl-3 text-[13px] leading-9 text-[var(--ink)] outline-none transition-colors focus-visible:border-ring focus-visible:ring-3 focus-visible:ring-ring/50 disabled:cursor-not-allowed disabled:opacity-50"
|
||||||
>
|
>
|
||||||
{modelOptions.map((model) => (
|
{modelOptions.map((model) => (
|
||||||
<option key={model.id} value={model.id}>
|
<option key={model.id} value={model.id}>
|
||||||
|
|||||||
@@ -29,6 +29,9 @@ test("assistant chat smoke after opt-in", async ({ page }) => {
|
|||||||
const modelSelector = page.getByRole("combobox", { name: "Assistant model" });
|
const modelSelector = page.getByRole("combobox", { name: "Assistant model" });
|
||||||
await expect(modelSelector).toBeVisible();
|
await expect(modelSelector).toBeVisible();
|
||||||
await expect.poll(() => modelSelector.evaluate((node) => node.tagName)).toBe("SELECT");
|
await expect.poll(() => modelSelector.evaluate((node) => node.tagName)).toBe("SELECT");
|
||||||
|
await expect
|
||||||
|
.poll(() => modelSelector.evaluate((node) => node.getBoundingClientRect().height))
|
||||||
|
.toBeGreaterThanOrEqual(36);
|
||||||
|
|
||||||
await page.getByLabel("Message for Assistant").fill("hello assistant");
|
await page.getByLabel("Message for Assistant").fill("hello assistant");
|
||||||
await page.getByRole("button", { name: "Send" }).click();
|
await page.getByRole("button", { name: "Send" }).click();
|
||||||
|
|||||||
Reference in New Issue
Block a user