hoverpulse icon indicating copy to clipboard operation
hoverpulse copied to clipboard

Updated version to allow for them to work in table cells

Open scottadamsmith opened this issue 13 years ago • 2 comments

I was trying to use this plugin with the images being individually placed in the a left most table cell for multiple rows, but ran into some issues because of the fact that they were no longer contained in a block style element. At first, I made an update the changed the table cell to a block, but throws off all the original styling of the table and how the cell aligns with the rest of the cells. Then I decided to just wrap the img in a div in the case that the parent container is a table cell and it seems to work like a charm.

scottadamsmith avatar Jun 07 '12 22:06 scottadamsmith

I added a new revision, this one more major than the previous. In my use case, my images are all different aspect ratios and slightly different sizes. I didn't want to initialize hoverpulse individually for each image, so I added a new option Magnify that can be used in place of the size and size_y options. See commit note for more detail. Let me know if you have any feedback.

scottadamsmith avatar Jun 08 '12 15:06 scottadamsmith

Nostalgic PR for me.

scottadamsmith avatar Jul 15 '20 04:07 scottadamsmith