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

Column pinning selection opacity issue #118

Open
1 task done
achennuru opened this issue Aug 29, 2023 · 5 comments
Open
1 task done

Column pinning selection opacity issue #118

achennuru opened this issue Aug 29, 2023 · 5 comments
Labels
V1 Issue with MRT V1

Comments

@achennuru
Copy link

mantine-react-table version

1.2.0

react & react-dom versions

18.2.0

Describe the bug and the steps to reproduce it

When we pin the colums to left or right and there are columns behind with row selection enabled the opacity of hover and selected row will make it so that the text behind the pinned columns overlap.

image

Minimal, Reproducible Example - (Optional, but Recommended)

Attached the screenshot and the code is below.

`
import { useState } from 'react';
import { MantineReactTable, useMantineReactTable } from 'mantine-react-table';
import { columns, data as initialData } from './makeData';
import { ActionIcon, Box } from '@mantine/core';
import { IconEdit, IconSend, IconTrash } from '@tabler/icons-react';

export const Example = () => {
const [data, setData] = useState(initialData);

const table = useMantineReactTable({
columns,
data,
enableRowSelection: true,
enableRowActions: true,
enablePinning: true,
renderRowActions: ({ row }) => (
<Box sx={{ display: 'flex', flexWrap: 'nowrap', gap: '8px' }}>
<ActionIcon
color="blue"
onClick={() =>
window.open(
mailto:kevinvandy@mailinator.com?subject=Hello ${row.original.firstName}!
)
}
>


<ActionIcon
color="orange"
onClick={() => {
table.setEditingRow(row);
}}
>


<ActionIcon
color="red"
onClick={() => {
data.splice(row.index, 1); //assuming simple data table
setData([...data]);
}}
>



),
});

return ;
};

export default Example;

`

Screenshots or Videos (Optional)

No response

Do you intend to try to help solve this bug with your own PR?

None

Terms

  • I understand that if my bug cannot be reliably reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.
@alessandrojcm alessandrojcm added the V1 Issue with MRT V1 label Dec 9, 2023
@ddanielcruzz
Copy link

This also happens in V2, here's a video from the documentation website:

Screen.Recording.2023-12-21.at.10.07.45.mov

I'm happy to help fix this if you can point what files should I check to debug it.

@alessandrojcm
Copy link
Collaborator

This was fixed for V2 in #229 although it's not published. We will see about back porting this fix for V1.

@nairvishal
Copy link

nairvishal commented Jan 17, 2024

@alessandrojcm any info on porting the fix to V1 yet ? or can you suggest any temporary fix we could use on our side until the porting happens.

@alessandrojcm
Copy link
Collaborator

@nairvishal I can take a look into this later today.

@alessandrojcm
Copy link
Collaborator

#247

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
V1 Issue with MRT V1
Projects
None yet
Development

No branches or pull requests

4 participants