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

fix: override isLoading to return proper value (#241) #303

Merged
merged 1 commit into from
Oct 26, 2020

Conversation

tomivirkki
Copy link
Member

Cherry-pick: #241

Web-component: grid

Fixes: vaadin/vaadin-grid#1809

Details: Flow Grid's connector overrides the ensureSubCacheForScaledIndex function, and adds a debouncer for the logic, the data request gets made asynchronously when expanding the Grids item. If the request for recalculateColumnWithds is done on expand listener, the width calculation might occasionally happen before the node gets expanded on the client-side, thus resulting into columns with undesired widths.

Web-component: grid

Fixes: vaadin/vaadin-grid#1809

Details: Flow Grid's connector overrides the ensureSubCacheForScaledIndex function, and adds a debouncer for the logic, the data request gets made asynchronously when expanding the Grids item. If the request for recalculateColumnWithds is done on expand listener, the width calculation might occasionally happen before the node gets expanded on the client-side, thus resulting into columns with undesired widths.
@tomivirkki tomivirkki merged commit 31fe9c1 into 17.0 Oct 26, 2020
@tomivirkki tomivirkki deleted the cherrypick-241-17 branch October 26, 2020 11:34
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

Successfully merging this pull request may close these issues.

None yet

2 participants