Skip to content

Adjust MCP server table light theme & localization additions #34

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
a-effort wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Adjust MCP server table light theme & localization additions #34

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 12 additions & 12 deletions e2e/resources.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -156,7 +156,7 @@ test.describe("Resources page", () => {

await page.getByRole("button", { name: "More options for github-server" }).click();

await expect(page.getByRole("menuitem", { name: "View Details" })).toBeVisible();
await expect(page.getByRole("menuitem", { name: "View details" })).toBeVisible();
});

test("truncates a long gateway name and keeps the overflow menu visible", async ({ page }) => {
Expand Down Expand Up @@ -197,7 +197,7 @@ test.describe("Resources page", () => {
expect(kebabBox!.x + kebabBox!.width).toBeLessThanOrEqual(cardBox!.x + cardBox!.width + 1);

await kebab.click();
await expect(page.getByRole("menuitem", { name: "View Details" })).toBeVisible();
await expect(page.getByRole("menuitem", { name: "View details" })).toBeVisible();
});

test("opens details panel when View Details is clicked", async ({ page }) => {
Expand All @@ -213,7 +213,7 @@ test.describe("Resources page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "More options for github-server" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();

const panel = page.getByRole("region", { name: /Resources for github-server/i });
await expect(panel).toBeVisible();
Expand All @@ -235,7 +235,7 @@ test.describe("Resources page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "More options for github-server" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();

const panel = page.getByRole("region", { name: /Resources for github-server/i });
await expect(panel).toBeVisible();
Expand All @@ -259,7 +259,7 @@ test.describe("Resources page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "More options for github-server" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();

const panel = page.getByRole("region", { name: /Resources for github-server/i });
await panel.getByRole("button", { name: "More options" }).first().click();
Expand Down Expand Up @@ -302,7 +302,7 @@ test.describe("Resources page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "More options for github-server" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();

const panel = page.getByRole("region", { name: /Resources for github-server/i });
await expect(panel).toBeVisible();
Expand Down Expand Up @@ -350,7 +350,7 @@ test.describe("Resources page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "More options for github-server" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();

const panel = page.getByRole("region", { name: /Resources for github-server/i });
await expect(panel).toBeVisible();
Expand Down Expand Up @@ -394,7 +394,7 @@ test.describe("Resources page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "More options for solo-gateway" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();

const panel = page.getByRole("region", { name: /Resources for solo-gateway/i });
await expect(panel).toBeVisible();
Expand Down Expand Up @@ -441,7 +441,7 @@ test.describe("Resources page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "More options for multi-gw" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();

const panel = page.getByRole("region", { name: /Resources for multi-gw/i });
await expect(panel).toBeVisible();
Expand Down Expand Up @@ -491,7 +491,7 @@ test.describe("Resources page", () => {
await expect(page.getByText("github-server")).toBeVisible();

await page.getByRole("button", { name: "More options for lone-gateway" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();

const panel = page.getByRole("region", { name: /Resources for lone-gateway/i });
await expect(panel).toBeVisible();
Expand Down Expand Up @@ -546,7 +546,7 @@ test.describe("Resources page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "More options for github-server" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();

const panel = page.getByRole("region", { name: /Resources for github-server/i });
await expect(panel).toBeVisible();
Expand Down Expand Up @@ -602,7 +602,7 @@ test.describe("Resources page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "More options for github-server" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();

const panel = page.getByRole("region", { name: /Resources for github-server/i });
await panel.getByRole("button", { name: "More options" }).first().click();
Expand Down
12 changes: 6 additions & 6 deletions e2e/servers.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -173,7 +173,7 @@ test.describe("MCP Servers page", () => {

await page.getByRole("button", { name: "Actions for GitHub MCP Server" }).click();

await expect(page.getByRole("menuitem", { name: "View Details" })).toBeVisible();
await expect(page.getByRole("menuitem", { name: "View details" })).toBeVisible();
await expect(page.getByRole("menuitem", { name: "Edit" })).toBeVisible();
await expect(page.getByRole("menuitem", { name: "Deactivate" })).toBeVisible();
await expect(page.getByRole("menuitem", { name: "Delete" })).toBeVisible();
Expand Down Expand Up @@ -336,7 +336,7 @@ test.describe("MCP Servers page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "Actions for GitHub MCP Server" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();
await expect(page.getByRole("heading", { name: "GitHub MCP Server" })).toBeVisible();
await page.keyboard.press("Escape"); // drawer is a full-height overlay; must close it before clicking table row actions
await expect(page.getByRole("button", { name: /close mcp server details/i })).not.toBeVisible();
Expand Down Expand Up @@ -402,7 +402,7 @@ test.describe("MCP Servers page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "Actions for GitHub MCP Server" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();
await expect(page.getByRole("heading", { name: "GitHub MCP Server" })).toBeVisible();
await page.keyboard.press("Escape"); // drawer is a full-height overlay; must close it before clicking table row actions
await expect(page.getByRole("button", { name: /close mcp server details/i })).not.toBeVisible();
Expand Down Expand Up @@ -469,7 +469,7 @@ test.describe("MCP Servers page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "Actions for GitHub MCP Server" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();
await expect(page.getByRole("heading", { name: "GitHub MCP Server" })).toBeVisible();
await page.keyboard.press("Escape"); // drawer is a full-height overlay; must close it before clicking table row actions
await expect(page.getByRole("button", { name: /close mcp server details/i })).not.toBeVisible();
Expand All @@ -484,7 +484,7 @@ test.describe("MCP Servers page", () => {
await expect(page.getByRole("row").filter({ hasText: "Slack MCP Server" })).not.toBeVisible();

await page.getByRole("button", { name: "Actions for GitHub MCP Server" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();
await expect(page.getByRole("heading", { name: "GitHub MCP Server" })).toBeVisible();
});

Expand Down Expand Up @@ -531,7 +531,7 @@ test.describe("MCP Servers page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "Actions for GitHub MCP Server" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();

// Wait for the details panel to open (heading "GitHub MCP Server" appears in the panel header)
await expect(page.getByRole("heading", { name: "GitHub MCP Server" })).toBeVisible();
Expand Down
32 changes: 16 additions & 16 deletions e2e/tools.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -225,7 +225,7 @@ test.describe("Tools page", () => {

// And it remains functional.
await kebab.click();
await expect(page.getByRole("menuitem", { name: "View Details" })).toBeVisible();
await expect(page.getByRole("menuitem", { name: "View details" })).toBeVisible();
});

test("opens more options dropdown and shows View Details item", async ({ page }) => {
Expand All @@ -242,7 +242,7 @@ test.describe("Tools page", () => {

await page.getByRole("button", { name: "More options for github-server" }).click();

await expect(page.getByRole("menuitem", { name: "View Details" })).toBeVisible();
await expect(page.getByRole("menuitem", { name: "View details" })).toBeVisible();
});

test("opens details panel when View Details is clicked", async ({ page }) => {
Expand All @@ -258,7 +258,7 @@ test.describe("Tools page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "More options for github-server" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();

const panel = page.getByRole("region", { name: /Tools for github-server/i });
await expect(panel).toBeVisible();
Expand All @@ -280,7 +280,7 @@ test.describe("Tools page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "More options for github-server" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();

const panel = page.getByRole("region", { name: /Tools for github-server/i });
await expect(panel).toBeVisible();
Expand Down Expand Up @@ -315,7 +315,7 @@ test.describe("Tools page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "More options for github-server" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();

const panel = page.getByRole("region", { name: /Tools for github-server/i });
await expect(panel).toBeVisible();
Expand Down Expand Up @@ -366,7 +366,7 @@ test.describe("Tools page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "More options for github-server" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();

const panel = page.getByRole("region", { name: /Tools for github-server/i });
await expect(panel).toBeVisible();
Expand Down Expand Up @@ -399,7 +399,7 @@ test.describe("Tools page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "More options for github-server" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();

const panel = page.getByRole("region", { name: /Tools for github-server/i });
await panel.getByRole("button", { name: "More options" }).first().click();
Expand Down Expand Up @@ -438,7 +438,7 @@ test.describe("Tools page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "More options for solo-gateway" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();

const panel = page.getByRole("region", { name: /Tools for solo-gateway/i });
await expect(panel).toBeVisible();
Expand Down Expand Up @@ -485,7 +485,7 @@ test.describe("Tools page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "More options for multi-gw" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();

const panel = page.getByRole("region", { name: /Tools for multi-gw/i });
await expect(panel).toBeVisible();
Expand Down Expand Up @@ -535,7 +535,7 @@ test.describe("Tools page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "More options for rb-gateway" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();

const panel = page.getByRole("region", { name: /Tools for rb-gateway/i });
await expect(panel).toBeVisible();
Expand Down Expand Up @@ -583,7 +583,7 @@ test.describe("Tools page", () => {
await expect(page.getByText("github-server")).toBeVisible();

await page.getByRole("button", { name: "More options for lone-gateway" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();

const panel = page.getByRole("region", { name: /Tools for lone-gateway/i });
await expect(panel).toBeVisible();
Expand Down Expand Up @@ -845,7 +845,7 @@ test.describe("Tools page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "More options for edit-gw" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();

const panel = page.getByRole("region", { name: /Tools for edit-gw/i });
await expect(panel).toBeVisible();
Expand Down Expand Up @@ -885,7 +885,7 @@ test.describe("Tools page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "More options for schema-gw" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();

const panel = page.getByRole("region", { name: /Tools for schema-gw/i });
await expect(panel).toBeVisible();
Expand Down Expand Up @@ -936,7 +936,7 @@ test.describe("Tools page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "More options for toggle-gw" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();

const panel = page.getByRole("region", { name: /Tools for toggle-gw/i });
await expect(panel).toBeVisible();
Expand Down Expand Up @@ -985,7 +985,7 @@ test.describe("Tools page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "More options for activate-gw" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();

const panel = page.getByRole("region", { name: /Tools for activate-gw/i });
await expect(panel).toBeVisible();
Expand Down Expand Up @@ -1034,7 +1034,7 @@ test.describe("Tools page", () => {
await page.waitForLoadState("networkidle");

await page.getByRole("button", { name: "More options for fail-gw" }).click();
await page.getByRole("menuitem", { name: "View Details" }).click();
await page.getByRole("menuitem", { name: "View details" }).click();

const panel = page.getByRole("region", { name: /Tools for fail-gw/i });
await expect(panel).toBeVisible();
Expand Down
Loading
Loading