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

text selection mixin now works in grid #247

Merged
merged 2 commits into from
Nov 20, 2020

Conversation

vaadin-miki
Copy link
Owner

closes #243

client code now asks server to reinitialise if it is not ready; this seems to require one or two round-trips to the server when the component is placed in a grid, but gets the job done

@vaadin-miki vaadin-miki self-assigned this Nov 20, 2020
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@vaadin-miki vaadin-miki merged commit 0387111 into development Nov 20, 2020
@vaadin-miki vaadin-miki deleted the 243-text-selection-mixin-grid branch November 20, 2020 22:27
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.

Text selection mixin throws JS errors when used in Grid
1 participant