Skip to content
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.

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

Babel's helpers are needed but not included in the build #80

Open
nskazki opened this issue Jan 21, 2025 · 1 comment
Open

Babel's helpers are needed but not included in the build #80

nskazki opened this issue Jan 21, 2025 · 1 comment

Comments

@nskazki
Copy link

nskazki commented Jan 21, 2025

The build needs regeneratorRuntime but doesn't include it, please either bundle it in or get rid of it. Otherwise, the package is great!

From a11y-vue-dialog/dist/a11y-vue-dialog.esm.js:218

dleOpen: function handleOpen() {
      var hasRefs;
      return regeneratorRuntime.async(function handleOpen$(_context) {
        while (1) {
          switch (_context.prev = _context.next) {
            case 0:
              _context.next = 2;
              return regeneratorRuntime.awrap(this.$nextTick());

            case 2:
              _context.next = 4;
              return regeneratorRuntime.awrap(this.$nextTick());

            case 4:
              _context.next = 6;
              return regeneratorRuntime.awrap(this.$nextTick());

            case 6:
              hasRefs = this.getDOMRefs(); // do no perform DOM actions if no DOM references

              if (hasRefs) {
                _context.next = 9;
                break;
              }

              return _context.abrupt("return");

            case 9:
              this.toggleFocusTrap(true);
              this.lookForSiblings();
              this.toggleVisibilityEvents(true);
              this.toggleContentAriaAttrs(true);

            case 13:
            case "end":
              return _context.stop();
          }
        }
      }, null, this);
    },
@renatodeleao
Copy link
Owner

renatodeleao commented Jan 22, 2025

Thanks @nskazki, don't even remember why i'm using, probably from some old IE compatibility or something. Will remove this thing in the next patch!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants