In #2479303: Expand the scope of this module, we added support for sticky columns to a few more Views styles. (Thanks for committing, by the way!)
Unfortunately, I overlooked something in the Views Matrix integration – the top-left cell doesn't get the sticky-column class, garbling the column.

CommentFileSizeAuthor
#2 2505865-2--small_bugfixes.patch1.03 KBdrunken monkey
#1 2505865-1--views_matrix_bugfix.patch617 bytesdrunken monkey

Comments

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Active » Needs review
StatusFileSize
new 2505865-1--views_matrix_bugfix.patch617 bytes

The attached patch fixes this. Would be great if you could commit this, too.
And sorry for the inconvenience!

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Title: Small bug in Views Matrix integration » Small bugs in new code
StatusFileSize
new 2505865-2--small_bugfixes.patch1.03 KB

Not a bug per se, but the id attribute should of course be unique in each HTML document, so we have to remove that when cloning the table.
Revised patch attached. (Since the changes are so small, I just put it in here. I can also create a separate issue, if you prefer.)

herd45’s picture

herd45 commented

I've tested and this works well

adci_contributor’s picture

adci_contributor commented

Hello everyone!
Sorry for long inactivity on this issue, there was a problem with access to module's Git. Patch by Thomas applied, thanks.

adci_contributor’s picture

adci_contributor commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.