mui-x icon indicating copy to clipboard operation
mui-x copied to clipboard

[data grid] range styling is being added to cells inside of the range instead of just the edges of the range

Open RyleySill93 opened this issue 1 year ago • 4 comments

Steps to reproduce

Link to live example: https://mui.com/x/react-data-grid/cell-selection/#customize-range-styling

image

Current behavior

MuiDataGrid-cell--rangeBottom is being applied to a cell in the middle of the range.

Expected behavior

MuiDataGrid-cell--rangeBottom should only be applied to cells in the last row of the range.

Context

No response

Your environment

npx @mui/envinfo
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Search keywords: datagrid range styling Order ID: 75803

RyleySill93 avatar Jan 31 '24 21:01 RyleySill93

Hey @RyleySill93 could you provide detailed steps you took to get to this bug? If I try it is always correct:

Screenshot 2024-02-01 at 12 06 38

Thanks! 🙇🏼

michelengelen avatar Feb 01 '24 11:02 michelengelen

@michelengelen I'm just clicking on that link and highlighting a range of cells using click and drag or shift + arrow keys. Tried with Chrome, Safari, and Firefox

https://github.com/mui/mui-x/assets/20671872/3359dbdf-8853-48d3-aed9-42399f296c71

RyleySill93 avatar Feb 01 '24 18:02 RyleySill93

Oh, yes ... now I can reproduce as well ... it happens with all of the range classes. Thanks for raising this issue. 🙇🏼 We will have a look.

michelengelen avatar Feb 02 '24 09:02 michelengelen

@michelengelen @RyleySill93 we would like to pick this up

gitstart avatar Feb 05 '24 08:02 gitstart

We're experiencing this as well and are planning to roll out our grid-based feature in the next two weeks. @michelengelen are you able to provide an estimated timeline for this one? If beyond 2-3 weeks we'll probably go ahead and create our own internal fix. Thanks so much!

Support Key/Order ID: 85118

louisaaron avatar Feb 26 '24 22:02 louisaaron

Which grid version will you be using? Can you use the v7 releases? This should be straightforward to fix this week.

romgrk avatar Feb 27 '24 07:02 romgrk

:warning: This issue has been closed. If you have a similar problem, please open a new issue and provide details about your specific problem. If you can provide additional information related to this topic that could help future readers, please feel free to leave a comment.

How did we do @RyleySill93? Your experience with our support team matters to us. If you have a moment, please share your thoughts through our brief survey.

github-actions[bot] avatar Feb 27 '24 17:02 github-actions[bot]

Thanks guys!

louisaaron avatar Feb 27 '24 20:02 louisaaron