Skip to content

Fix Blur #170

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.

Sign up for GitHub

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

Jump to bottom
Merged
iddan merged 2 commits into from
Sep 23, 2021
Merged

Fix Blur #170

iddan merged 2 commits into from
Sep 23, 2021

Conversation

@iddan
Copy link
Owner

@iddan iddan commented Sep 23, 2021 •
edited
Loading

Resolves #153
Resolves #118
Blur spreadsheet (clear active and selected) when the component loses focus

@iddan iddan added the bug Something isn't working label Sep 23, 2021
@iddan iddan self-assigned this Sep 23, 2021
@coveralls
Copy link

coveralls commented Sep 23, 2021 •
edited
Loading

Pull Request Test Coverage Report for Build 1265404697

  • 8 of 14 (57.14%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.8%) to 79.313%
Changes Missing Coverage Covered Lines Changed/Added Lines %
src/Spreadsheet.tsx 7 13 53.85%
Totals Coverage Status
Change from base Build 1265091172: -0.8%
Covered Lines: 715
Relevant Lines: 847
💛 - Coveralls

@iddan iddan merged commit e453652 into master Sep 23, 2021
@iddan iddan deleted the fix/blur branch September 23, 2021 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@iddan iddan

Labels

bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Handle inactive state of the component remove focus on spreadsheet cell

3 participants

@iddan @coveralls