eldargab / easy-table

Nice text table for Node.js
314 stars 31 forks source link

Sort shouldn't be case sensitive #27

Open gricey432 opened 3 years ago

gricey432 commented 3 years ago

Not sure if this is desired behaviour, but it seems a bit odd to me.

Source:

var Table = require('easy-table')

var data = [
    {name: "Amy", snack: 'Apple'},
    {name: "Bob", snack: 'apple'},
    {name: "Charles", snack: 'Banana'},
    {name: "Dana", snack: 'Apple'},
    {name: "Eric", snack: 'Zuchini'},
]

var t = new Table

data.forEach(function(r) {
    t.cell('Person', r.name)
    t.cell('Snack', r.snack)
    t.newRow()
})
t.sort(['Snack'])
console.log(t.toString())

Output:

Person   Snack
-------  -------
Amy      Apple
Dana     Apple
Charles  Banana
Eric     Zuchini
Bob      apple

Expected:

Person   Snack
-------  -------
Amy      Apple
Bob      apple
Dana     Apple
Charles  Banana
Eric     Zuchini