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: error when edited cell is out of grid viewport #2148

Merged
merged 3 commits into from
Jul 17, 2024

Conversation

AkshatJawne
Copy link
Contributor

@AkshatJawne AkshatJawne commented Jul 16, 2024

Resolves #2087

Steps Moving Forward:

  • Clarify whether we want to be able to handle edits that are not in the current viewport -- with my understanding, this will mean we will have to load in more grid data, which does not seem overly valuable (in terms of the efficiency tradeoff) since we are dealing with an improbable edge case where the user begins to edit in the viewport, continues editing outside of the viewport where the cell originally was, and then wants to commit those changes outside as well

** A better solution for the edge case above IMO is to create a new enhancement ticket where we attempt to have the editing cell be floating/sticky in the grid viewport, similar to how it is done on Google Sheets **

Ticket Created: #2153

Screenshot 2024-07-16 at 10 37 02 AM

@AkshatJawne AkshatJawne requested a review from mofojed July 16, 2024 16:39
@AkshatJawne AkshatJawne self-assigned this Jul 16, 2024
Copy link

codecov bot commented Jul 16, 2024

Codecov Report

Attention: Patch coverage is 71.42857% with 2 lines in your changes missing coverage. Please review.

Project coverage is 46.62%. Comparing base (0da547d) to head (9fd7a75).
Report is 2 commits behind head on main.

Files Patch % Lines
packages/grid/src/Grid.tsx 71.42% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2148      +/-   ##
==========================================
- Coverage   46.62%   46.62%   -0.01%     
==========================================
  Files         685      685              
  Lines       38492    38497       +5     
  Branches     9800     9592     -208     
==========================================
+ Hits        17948    17950       +2     
- Misses      20492    20537      +45     
+ Partials       52       10      -42     
Flag Coverage Δ
unit 46.62% <71.42%> (-0.01%) ⬇️

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

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

Copy link
Member

@mofojed mofojed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small cleanup.

Feel free to file an feature request for the floating google docs type behaviour you've described. With the floating cell we'd want to display the cell coordinates next to the floating input, but that could get cluttered with a long column name, and also we don't display the row headers by default. Just to note on the ticket.

image

packages/grid/src/Grid.tsx Outdated Show resolved Hide resolved
@AkshatJawne AkshatJawne requested a review from mofojed July 17, 2024 16:32
@AkshatJawne AkshatJawne merged commit 3fccd43 into deephaven:main Jul 17, 2024
12 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jul 17, 2024
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.

Error when Edited Cell scrolls out of Viewport
2 participants