Skip to content

Commit

Permalink
Add note of comma bug from Michel R
Browse files Browse the repository at this point in the history
  • Loading branch information
stuartlangridge committed Jan 22, 2014
1 parent 4bdfae5 commit 6097933
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions PLANS
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
Planned things to do in sorttable v3

Full test suite
Publish test suite and make it easy for someone to run it and report back
Track runs of the test suite and record data (google analytics?)
Maintain "does not need a library"
Maintain "just include sorttable and it works without configuration"
Maintain compatibility back to IE6
Expand All @@ -16,5 +18,9 @@ Provide debug console to see what sorttable is doing
Make JSHint clean
Obfuscate </script> in comment to avoid breaking older Firefoxes.
Replace literal high-bit-set characters in regexp with escapes to avoid unicode hassle: if (text.match(/^-?[\xA2\xA3\xA5\u20A5\u20B9\u20AC\u20A1\u20B5\x24]?[\d,.]+%?$/)) or similar
Work when there are more then 256 rows

Work when there are more then 256 rows (added to test suite)
Accessibility stuff
Fix getinnertext to work with many different types of content (unicode, html, emptiness, text)
Fix mixed content warning in I by not using javascript:void as link dest: use :// instead or similar
Numeric comparisons are the wrong way around: should be bb-aa not aa-bb
A cell with a single "," in it makes sorttable diagnose the column as containing floats. See email from Michel R

0 comments on commit 6097933

Please sign in to comment.