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

Using the popup fails with Vaadin 22 - TypeError: popupTheme.register is not a function #14

Open
karstenlehmann opened this issue Dec 3, 2021 · 2 comments

Comments

@karstenlehmann
Copy link

It seems that the current version of the popup fails with Vaadin 22.0.0. In the vaadin log we got an

TypeError: popupTheme.register is not a function

The browser (chrome) console logs the following error:

Uncaught (in promise) TypeError: popupTheme.register is not a function at eval (vcf-popup-styles.js?4b11:59) at Module.../node_modules/.pnpm/@vaadin-component-factory/[email protected]/node_modules/@vaadin-component-factory/vcf-

@karstenlehmann
Copy link
Author

Simple Test case to reproduce the error

@Route("")
public class MainView
    extends VerticalLayout {

  public MainView() {

    Grid<String> grid = new Grid<>();

    Label header = new Label("My header with popup");
    String uid = "ExtendedGridHeader" + UUID.randomUUID().toString(); //$NON-NLS-1$
    header.setId(uid);

    // register popup
    Popup p = new Popup();
    p.setFor(uid);
    add(p);

    grid.addColumn(obj -> "Test").setHeader(header);

    add(grid);
  }

}

After that you have to start the test in production mod - the dev mod should work

e.g. with Maven: -C clean package -Pproduction jetty:run

@bwajtr
Copy link
Contributor

bwajtr commented May 23, 2023

Please try with version 3.0.0 for Vaadin 22, should be fixed there.
Please use version 23.3.0 if you are targetting Vaadin 23

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