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

feat: Update go to row panel's row number with cursorRow #1508

Merged
merged 6 commits into from
Sep 14, 2023

Conversation

georgecwan
Copy link
Contributor

@georgecwan georgecwan commented Sep 12, 2023

The row number in the go to row panel now updates when users click on a row in the table. The value displayed when multiple selections or ranges of rows are selected depends on the value of cursorRow.

Fixes a bug where selecting a row then using the context menu to open the go to row panel would fill in the panel with the incorrect row number.

Resolves #1406.

@georgecwan georgecwan self-assigned this Sep 12, 2023
@codecov
Copy link

codecov bot commented Sep 12, 2023

Codecov Report

Patch coverage: 57.14% and project coverage change: -0.01% ⚠️

Comparison is base (32e6d20) 46.42% compared to head (ca1a217) 46.42%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1508      +/-   ##
==========================================
- Coverage   46.42%   46.42%   -0.01%     
==========================================
  Files         558      558              
  Lines       35706    35719      +13     
  Branches     8917     8921       +4     
==========================================
+ Hits        16576    16582       +6     
- Misses      19080    19087       +7     
  Partials       50       50              
Flag Coverage Δ
unit 46.42% <57.14%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
packages/iris-grid/src/IrisGrid.tsx 26.96% <25.00%> (-0.01%) ⬇️
packages/iris-grid/src/GotoRow.tsx 58.87% <58.41%> (-0.31%) ⬇️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

packages/iris-grid/src/IrisGrid.tsx Show resolved Hide resolved
packages/iris-grid/src/GotoRow.tsx Outdated Show resolved Hide resolved
packages/iris-grid/src/GotoRow.tsx Outdated Show resolved Hide resolved
@georgecwan georgecwan enabled auto-merge (squash) September 14, 2023 15:36
@georgecwan georgecwan merged commit 23ab5cc into deephaven:main Sep 14, 2023
5 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Sep 14, 2023
@georgecwan georgecwan deleted the 1406-go-to-row-panel-number branch September 14, 2023 15:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Go to row panel should update the row number as you navigate the selection
2 participants