chore: cleanup

Author: sheremet-va

docs/api/browser/locators.md

@@ -938,11 +938,13 @@ page.getByText('Hello USA').elements() // ✅ []
 ### waitForElement <Version>4.1.0</Version> {#waitforelement}
 
 ```ts
-function waitForElement(options?: SelectorOptions): Promise<HTMLElement | SVGElement>
+function waitForElement(
+  options?: SelectorOptions
+): Promise<HTMLElement | SVGElement>
 ```
 
 ::: danger WARNING
-This is an escape hatch for library authors and 3d-party APIs that do not support locators directly. If you are interacting with the element, use [builtin methods](#methods) instead.
+This is an escape hatch for library authors and 3d-party APIs that do not support locators directly. If you are interacting with the element, use other [builtin methods](#methods) instead.
 :::
 
 This method returns an element matching the locator. Unlike [`.element()`](#element), this method will wait and retry until a matching element appears in the DOM, using increasing intervals (0, 20, 50, 100, 100, 500ms).
@@ -953,7 +955,7 @@ If _multiple elements_ match the selector and `strict` is `true` (the default),
 
 It accepts options:
 
-- `timeout: number` - How long to wait in milliseconds until a single element is found. By default, this has the same timeout as the test.
+- `timeout: number` - How long to wait in milliseconds until at least one element is found. By default, this shares timeout with the test.
 - `strict: boolean` - When `true` (default), throws an error if multiple elements match the locator. When `false`, returns the first matching element.
 
 Consider the following DOM structure:

packages/browser/context.d.ts

@@ -715,7 +715,7 @@ export interface Locator extends LocatorSelectors {
    * This is an escape hatch for library authors and 3d-party APIs that do not support locators directly.
    * If you are interacting with the element, use builtin methods instead.
    * @since 4.1.0
-   * @see {@link https://vitest.dev/api/browser/locators#waitForElement}
+   * @see {@link https://vitest.dev/api/browser/locators#waitforelement}
    */
   waitForElement(options?: SelectorOptions): Promise<HTMLElement | SVGElement>
 }

packages/browser/src/client/tester/context.ts

@@ -323,13 +323,16 @@ export const page: BrowserPage = {
     const name
       = options.path || `${taskName.replace(/[^a-z0-9]/gi, '-')}-${number}.png`
 
+    const [element, ...mask] = await Promise.all([
+      options.element ? convertToSelector(options.element) : undefined,
+      ...('mask' in options
+        ? (options.mask as Array<Element | Locator>).map(convertToSelector)
+        : []),
+    ])
+
     const normalizedOptions = 'mask' in options
-      ? {
-          ...options,
-          mask: await Promise.all((options.mask as Array<Element | Locator>).map(convertToSelector)),
-        }
+      ? { ...options, mask }
       : options
-    const element = options.element ? await convertToSelector(options.element) : undefined
 
     return ensureAwaited(error => triggerCommand(
       '__vitest_screenshot',