Alpha testing of Vaadin 24.6.0.alpha3 vs new Flow features

[mshabarov]

Test the happy path of these new features:

• WebPush notification options
• setDragImage
• exclude Vaadin components
• easier config for SpringServlet
• Add support for customizable projectFileExtensions

[mshabarov]

For anyone willing to start this - I'd love to try and test:
-exclude Vaadin components
-easier config for SpringServlet

[caalador]

• Web push - ok
• Drag image - ok
• exclude Vaadin components - works, but no documentation
• easier config for SpringServlet - ok, no doc?
  Skipping file extensions as I did that feature.

[mshabarov]

vaadin/docs#3913

[mcollovati]

• setDragImage - ok
• easier config for SpringServlet - ok
• frontendExtraFileExtensions - ok, but seems to miss docs. Also see question below

Add support for customizable projectFileExtensions: tested and works when importing things like @CssImport("some.scss").
The new frontendExtraFileExtensions seems not to be documented in https://vaadin.com/docs/latest/flow/production/production-build.
A question: I tried set frontendExtraFileExtensions=boh and to import a file into a JS script included with @JsModule ont the view. The JavaScript contains import something from 'Frontend/my.boh?raw';. However, my.boh is not included in the stats JSON. Is this expected?

Side note, the maven plugin documentation seems a bit outdated. For example, it lists an includes setting that does not seem to exist.

[caalador]

I would expect it to be there if it is handled by vite.

As boh marked with ?raw vite imports is as a String and it is thus not handled as a file, mening that vite didn't handle it and it will not show up.

[mcollovati]

Got it. Thanks for clarification.

[mshabarov]

exclude Vaadin components
Does it's job for React and Lit based Vaadin apps.

However, I got these info messages, even though I've already excluded components and copilot:

        <dependency>
            <groupId>com.vaadin</groupId>
            <artifactId>vaadin</artifactId>
            <exclusions>
                <exclusion>
                    <groupId>com.vaadin</groupId>
                    <artifactId>vaadin-core-components</artifactId>
                </exclusion>
                <exclusion>
                    <groupId>com.vaadin</groupId>
                    <artifactId>copilot</artifactId>
                </exclusion>
            </exclusions>
        </dependency>

message:

2024-11-18T12:14:57.440+02:00  INFO 6446 --- [onPool-worker-2] c.v.f.server.frontend.TaskUpdateImports  :

  Failed to find the following imports in the `node_modules` tree:
      - @vaadin/list-box/src/vaadin-list-box.js
      - @vaadin/number-field/src/vaadin-number-field.js
      - @vaadin/grid/src/vaadin-grid-sorter.js
      - ....

See exclude-vaadin-components.txt

@tltv is this expected? Could we avoid these messages?

[mshabarov]

This would be great to resolve before beta #20490

[mshabarov]

vaadin/docs#3921

[tltv]

With <artifactId>vaadin</artifactId> it includes pro component dependencies, which afaik depends on core components. Try changing that to vaadin-core and it should work. This is documentation issue and need to clear that pro components need to be excluded too by using only vaadin-core dependency.

[mshabarov]

Thanks, why I couldn't resolve it by myself 😄
Nothing bothers me after I added this:

 <dependency>
            <groupId>com.vaadin</groupId>
            <artifactId>vaadin-core</artifactId>
            <exclusions>
                <exclusion>
                    <groupId>com.vaadin</groupId>
                    <artifactId>vaadin-core-components</artifactId>
                </exclusion>
                <exclusion>
                    <groupId>com.vaadin</groupId>
                    <artifactId>copilot</artifactId>
                </exclusion>
                <exclusion>
                    <groupId>com.vaadin</groupId>
                    <artifactId>vaadin-icons-flow</artifactId>
                </exclusion>
            </exclusions>
        </dependency>

These three exclusions have to be documented, I believe.