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

Grid with component renderer shows empty rows on chrome 72 #1563

Closed
bbaumgartner opened this issue Feb 15, 2019 · 3 comments
Closed

Grid with component renderer shows empty rows on chrome 72 #1563

bbaumgartner opened this issue Feb 15, 2019 · 3 comments

Comments

@bbaumgartner
Copy link

Description

Given:

    @Route("grid")
    public class GridView extends VerticalLayout {
    
        public GridView() {
            setSizeFull();
    
            Grid<Object> grid = new Grid<>();
    
            grid.addColumn(new ComponentRenderer<>(row -> {
                VerticalLayout result = new VerticalLayout();
    
                Label label = new Label("Row " + row.toString());
                result.add(label);
    
                return result;
            })).setHeader("Component Renderer");
    
            grid.setItems(IntStream.range(0, 500).boxed().collect(Collectors.toList()));
    
            add(grid);
        }
    }

Expected outcome

Show the grid with the rows:
screenshot 2019-02-15 at 08 21 01

Actual outcome

On chrome 72.0.3626.109:
screenshot 2019-02-15 at 08 22 18

Live Demo

Well, not exactly the same but I think it is related:

  1. Open with chrome 72: https://admin-ui-demo.comerge.net/card-view
  2. Switch to simple view
  3. Switch to card view
  • IS: The first card is Aaliyah Lohmann
  • SHOULD: The first card is Aaliyah Alizadeh
    Now resize the browser window until the first card magically pops up

example

Browsers Affected

As far as I see only Chrome 72 is affected, it works in all other browsers, including chrome 71 and chrome canary

@bbaumgartner
Copy link
Author

Oh yeah, that's with vaadin 12.0.6/ grid 2.1.5

@web-padawan
Copy link
Member

Duplicate of vaadin/flow#5025 which is already fixed in Flow. The fix to be backported to V12.
We understand the importance of this issue so the team will take care of landing the fix ASAP.

@bbaumgartner
Copy link
Author

Thank you Serhii

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