playwright-ui5 1.2.0 → 1.3.0

package/README.md

@@ -35,4 +35,5 @@ this selector engine uses css selector-like syntax. the main difference is that
 | attribute selectors | `[text]`, `[text='foo']`,`[text*='foo']`                | ✔        | some equality mods are useless for ui5 (eg. `\|=`) but are supported for the sake of completeness |
 | id selectors        | `sap.m.Button#foo`                                      | ✔        | you should not use id selectors if the id is generated (eg. `__button1`) as they can change often |
 | nesting             | `sap.m.Table sap.m.Button`,`sap.m.Table > sap.m.Button` | ❌       | use playwright selector nesting instead (`ui5=sap.m.Table >> ui5=sap.m.Button`)                   |
-| pseudo-classes      | `sap.m.Table:has(sap.m.Button)`                         | ✔        | `:has` is supported                                                                               |
+| pseudo-classes      | `sap.m.Table:has(sap.m.Button)`                         | ✔        | only `:has` is supported for now                                                                  |
+| selector lists      | `sap.m.Button,sap.m.Table`                              | ✔        |

package/dist/browser/main.js

@@ -1372,9 +1372,8 @@ var parse = (0, import_css_selector_parser.createParser)({
     classNames: true
   }
 });
-var queryRule = (root, selector) => {
+var querySelector = (root, selector) => selector.rules.flatMap((rule) => {
   var _a;
-  const rule = (0, import_throw_expression.throwIfUndefined)(selector.rules[0], "rules array was empty");
   if (((_a = rule.tag) == null ? void 0 : _a.type) === "TagName" && rule.classNames) {
     const sapNamespace = "sap";
     if (rule.tag.name !== sapNamespace) {
@@ -1416,10 +1415,10 @@ var queryRule = (root, selector) => {
     });
   });
   return controls.map((control) => control.getDomRef()).filter((element) => {
-    if (element === null || root.querySelector(`#${element.id}`) === null) {
+    if (element === null || root.querySelector(`[id='${element.id}']`) === null) {
       return false;
     }
-    if (rule.pseudoClasses && queryRule(
+    if (rule.pseudoClasses && querySelector(
       element,
       (0, import_throw_expression.throwIfUndefined)(
         rule.pseudoClasses[0],
@@ -1430,19 +1429,16 @@ var queryRule = (root, selector) => {
     }
     return true;
   });
-};
+});
 var queryAll = (root, selector) => {
   const parsedSelector = parse(selector);
   if (selector === "") {
     throw new Error("ui5 selector is empty");
   }
-  if (parsedSelector.rules.length > 1) {
-    throw new Error("comma-separated selectors not supported");
-  }
   if (typeof sap === "undefined") {
     return [];
   }
-  return queryRule(root, parsedSelector);
+  return querySelector(root, parsedSelector);
 };
 var main_default = {
   queryAll,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "playwright-ui5",
-  "version": "1.2.0",
+  "version": "1.3.0",
   "files": [
     "dist",
     "src/node"