Simplify e2e: deterministic waits and locale-agnostic selectors

[Copilot]

E2E helpers relied on waitForTimeout fixed delays and locale-sensitive text selectors (has-text("Save"), has-text("Cancel")), causing flaky tests in non-English locales and under variable load.

Changes

• Deterministic loading guards (navigateToFolder, navigateToRoot, clearSearch): wait for .media-library-loading to disappear before returning instead of assuming the grid is ready.

• Locale-agnostic TinyMCE button selectors: replace text-based matching with CSS class structure:

  • Primary action: .tox-button:not(.tox-button--secondary):not(.tox-button--naked)
  • Cancel/dismiss: .tox-button--secondary with text-based fallbacks retained as safety net
  • Both paths add an Escape/Enter keyboard fallback and wait for dialog hidden state.

• Tighter nav click target in idevices-import-regression.spec.ts: use .node-text-span instead of .nav-element-text (the container also holds dropdown buttons).

• Replaced waitForTimeout(200) in the regression spec with a two-phase data-ready attribute poll on #node-content:

  // Wait for load to start (transition away from ready)
  await page.waitForFunction(
      () => document.querySelector('#node-content')?.getAttribute('data-ready') !== 'true',
      undefined, { timeout: 1000 }
  ).catch(() => {});
  // Then wait for load to complete
  await page.waitForFunction(
      () => document.querySelector('#node-content')?.getAttribute('data-ready') === 'true',
      undefined, { timeout: 10000 }
  ).catch(() => {});

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.