mathesar icon indicating copy to clipboard operation
mathesar copied to clipboard

In the table widget, column header cells should scroll underneath the origin cell

Open seancolsen opened this issue 6 months ago • 5 comments

Steps to reproduce

  1. Navigate to a record page for a record that has linked records.

  2. Within the linked record table widget, scroll the sheet horizontally.

  3. Expect the column header cells to scroll under header cell at the top-left (like they do on the table page).

  4. Instead, observe that the column header cells appear over that top-left cell as they scroll.

    image

I uncovered this during QA testing for the v0.1.4 release. I would classify this bug within a strange gray area between "regression" and "not regression". We had another bug in the previous release affecting horizontal scrolling, #3289. This Z-index issue is a bug that is now exposed due to the fixing of that other (much worse) bug. I put this in the v0.1.4 milestone so that we can try to fix it if we have time. But this would be an easy one to bump to a later release.

seancolsen avatar Jan 18 '24 01:01 seancolsen

Hey @seancolsen! While I was working on this issue. I saw this too: Screenshot 2024-01-22 at 3 24 12 PM I was curious if its intentional or its also a "z-index" bug too. Ig it's somewhat related to this issue also.

GopalVerma1303 avatar Jan 22 '24 09:01 GopalVerma1303

I've moved this out of the v0.1.4 milestone since it's small and we're short on time.

seancolsen avatar Jan 24 '24 16:01 seancolsen

@GopalVerma1303

I saw this too

That's #3380 (now fixed)

seancolsen avatar Feb 01 '24 18:02 seancolsen

To anyone working on this issue:

Note that this issue is regarding the table widget on the record page as shown here:

image

This issue is not present on the main table page.

seancolsen avatar Feb 01 '24 18:02 seancolsen

I took a look at this today while working in code nearby.

It's not a z-index issue. The problem is that the origin cell is not being made sticky like the row header cells. I briefly tried to fix it but got mildly perplexed. Our CSS appears as thoug it should be making that cell sticky. This is going to take some further troubleshooting to diagnose and fix.

seancolsen avatar Feb 06 '24 21:02 seancolsen