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.

Sign up for GitHub

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

Jump to bottom

Fixed failing sorts after multiple calls to makesortable and extended matches for percent values #11

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fixed failing sorts after multiple calls to makesortable and extended matches for percent values #11

wants to merge 2 commits into from

Commits on May 3, 2016

  1. Update sorttable.js 

    1. Fixed issue where sort would work only once for each column, after updating table content dynamically. Reason: multiple handlers were stacked if makesortable was called several times on same table (or when makesortable called before init).
2. Fixed issue where "num %" values would not be sorted correctly due to space chars before %.
3. Reverse sort activated by default (using mostly percentages).
    @virgile-hogman
    virgile-hogman committed May 3, 2016
    Configuration menu
    Copy the full SHA
    70b33ad View commit details
    Browse the repository at this point in the history

  2. Update sorttable.js 

    retro-fit current github version and remove reverse patch
    @virgile-hogman
    virgile-hogman committed May 3, 2016
    Configuration menu
    Copy the full SHA
    c5d4193 View commit details
    Browse the repository at this point in the history