meetselva / fixed-table-rows-cols

Fixed Table Header and Columns
http://meetselva.github.com/fixed-table-rows-cols/
Other
77 stars 42 forks source link

Heading of table contain two row #28

Closed mag2000 closed 8 years ago

mag2000 commented 8 years ago

I have table which have two rows of heading. The plugin is not working. Please if you have solution then let me know. sample of my table header

tbill2

meetselva commented 8 years ago

The plugin was not tested with multiple headers, but with some modifications we can make it work.

Could you post your html table in https://jsfiddle.net/ or somewhere I can look at?

mag2000 commented 8 years ago

https://jsfiddle.net/y7L6gd2t/

ddurose commented 8 years ago

I am also interested in having a header that spans two rows. Any chance you could implement this in your plug-in? Thanks for considering.

mag2000 commented 8 years ago

Waiting for your reply. Thanks for help

meetselva commented 8 years ago

Sorry about that, I did not had to change anything to make it work. I tried FF and IE Edge and it looked good. Test it in other browser and let me know.

https://jsfiddle.net/9jsb4oto/

mag2000 commented 8 years ago

Thanks for your reply. May be i made some mistake will check and give feed back

Best Regards

mag2000 commented 8 years ago

I check locally and it is working.

Thanks for your prompt reply

mag2000 commented 8 years ago

One problem with data. When we fixed the column it is not working. I change the value 1 to fixed column on your above code and it is not working

Please can you fixed this as well.

meetselva commented 8 years ago

I have made a change to handle multiple tr in thead. Below is the exact code changes,

https://github.com/meetselva/fixed-table-rows-cols/blob/master/js/fixed_table_rc.js#L124-L130

For now, grab the code from the above URL or fiddle https://jsfiddle.net/ccLfz0ox/ . I need to test this change and will promote it in the next release.

Please test from your end and let me know how it looks.

mag2000 commented 8 years ago

It is working fine. I graph from https://jsfiddle.net/ccLfz0ox/ net the code

mag2000 commented 8 years ago

Thanks for prompt reply and solution

mag2000 commented 8 years ago

I compared both the code and found only one difference in code. I check both and both working. Which one you recommend

.css({height: cfg.height - 17}) (Jsfiddle)

.css({height: lc.ft_container.height() - 17}) (Github)

meetselva commented 8 years ago

Use the one from Github. The jsFiddle one must be using the old JS.

omsysdesign commented 6 years ago

Thanks Buddy your solutions is worked for me. your work has too much appreciable.

many many thanks