Navigation Menu

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

Text wrap on verbose columns lost after sort. #84

Closed
bentwonk opened this issue Mar 29, 2015 · 10 comments
Closed

Text wrap on verbose columns lost after sort. #84

bentwonk opened this issue Mar 29, 2015 · 10 comments

Comments

@bentwonk
Copy link

Hi
Love the functionality of tablesort, but I have an issue with it sorting columns that require wordwrap,

When when a sort is applied to the initial result (beforeSort.png) the table re-formats so that each row is one line high, (afterSort.png) which has the effect of shoving half the table outside the viewing area.

I tried altering the table css, (table-layout: fixed; word-wrap: break-word;) etc, but now the text overwrites itself (withTableLayoutFixed.png)

Is it possible to sort, while preserving multi-line text in a column?

Cheers Ben

aftersort
beforesort
withtablelayoutfixed

@jhvst
Copy link
Collaborator

jhvst commented Mar 30, 2015

Mind pasting some of the code used in your table to jsfiddle or codepen? It would make debugging possible for us.

@bentwonk
Copy link
Author

Here is my little test case. I suspect its more to do with my css.

Also can linked text be embedded into the table? and be invisible to the
search?

Thank you

Ben

On Mon, Mar 30, 2015 at 4:00 PM, Juuso Haavisto notifications@github.com
wrote:

Mind pasting some of the code used in your table to jsfiddle or codepen?
It would make debugging possible for us.


Reply to this email directly or view it on GitHub
#84 (comment).

@jhvst
Copy link
Collaborator

jhvst commented Mar 30, 2015

@bentwonk are you sure you pasted any code? I can see none...

@bentwonk
Copy link
Author

ahh I sent you a zip, you have an email I can send it too?

On Mon, Mar 30, 2015 at 9:31 PM, Juuso Haavisto notifications@github.com
wrote:

@bentwonk https://github.com/bentwonk are you sure you pasted any code?
I can see none...


Reply to this email directly or view it on GitHub
#84 (comment).

@jhvst
Copy link
Collaborator

jhvst commented Mar 30, 2015

I'd really appreciate if you could paste your code to either jsfiddle or codepen.

@bentwonk
Copy link
Author

UI put my code up at
http://jsfiddle.net/#&togetherjs=7buL68vjsG

but I don't use jfiddle, so don't know how to include the tablesort.js file

On Mon, Mar 30, 2015 at 10:15 PM, Juuso Haavisto notifications@github.com
wrote:

I'd really appreciate if you could paste your code to either jsfiddle
https://jsfiddle.net/ or codepen http://codepen.io/.


Reply to this email directly or view it on GitHub
#84 (comment).

@jhvst
Copy link
Collaborator

jhvst commented Apr 2, 2015

Ugh, just re-send the zip file to my email address shown on my Github profile. The jsfiddle you sent me does not work.

@tristen
Copy link
Owner

tristen commented Apr 2, 2015

@bentwonk Can you simplify your test case and not use togetherjs? Just the html table with content and an initialization of Tablesort. This will make it easier to debug/fix.

You can add an external script in JSFiddle by selecting External Resources from the left sidebar.

@bentwonk
Copy link
Author

bentwonk commented Apr 2, 2015

Here you go

thanks
Ben

On Thu, Apr 2, 2015 at 11:36 PM, tristen notifications@github.com wrote:

@bentwonk https://github.com/bentwonk Can you simplify your test case
and not use togetherjs? Just the html table with content and an
initialization of Tablesort. This will make it easier to debug/fix.

You can add an external script in JSFiddle by selecting External
Resources
from the left sidebar.


Reply to this email directly or view it on GitHub
#84 (comment).

@tristen
Copy link
Owner

tristen commented May 20, 2015

Closing. Feel free to re-open @bentwonk if you can simplify your test case as outlined above

@tristen tristen closed this as completed May 20, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants