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

Suggestion: Use .html() instead of .text() #5

Closed
ghost opened this issue Feb 10, 2014 · 10 comments
Closed

Suggestion: Use .html() instead of .text() #5

ghost opened this issue Feb 10, 2014 · 10 comments
Assignees
Milestone

Comments

@ghost
Copy link

ghost commented Feb 10, 2014

In most cases we would want to get all links and form elements cascaded into table cells.

@micc83
Copy link
Owner

micc83 commented Feb 10, 2014

Hi @kaiga747, i'll keep it in mind for next update. I guess i'll make it optional, just in case 😄

@LubosRemplik
Copy link

+1

@micc83
Copy link
Owner

micc83 commented Oct 21, 2014

Hi @kaiga747 , @LubosRemplik
Would you like to check version 0.1.2 with keepHtml option to see if it fit your needs ?
https://github.com/micc83/ReStable/tree/v0.1.2

@micc83 micc83 added this to the 0.1.2 milestone Oct 21, 2014
@micc83 micc83 self-assigned this Oct 21, 2014
@jelewis8
Copy link

Image tags seem to get stripped out. I have keepHtml set to true.
Any ideas?

@micc83
Copy link
Owner

micc83 commented Feb 20, 2015

Using version 0.1.2? Could you make an example on http://jsfiddle.net/ ?

@jelewis8
Copy link

jelewis8 commented Mar 9, 2015

Sorry, been out of the loop, just now saw this!

Here's a fiddle of it: http://jsfiddle.net/96ap2c27/2/

@micc83
Copy link
Owner

micc83 commented Mar 11, 2015

Hello @jelewis8, check issue #8, it might be the same issue.

@jelewis8
Copy link

I tried to follow what the problem was in that other thread but couldn't make it out.
I'm using 0.1.2. Did you see my fiddle? http://jsfiddle.net/96ap2c27/2/

@jelewis8
Copy link

AHH, NM. Just set rowheader to false and that seems to fix the issue.

Though I'm still not sure why.

@micc83
Copy link
Owner

micc83 commented Mar 20, 2015

Hello @jelewis8, I'll make sure to check it out before releasing 0.1.2.
Thank you

@micc83 micc83 closed this as completed Oct 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants