11ty/eleventy-navigation

Change default `order` sort

TigersWay opened this issue · 4 comments

TigersWay commented

Hi,
Default order value = 0, means not-ordered items are going to be before the ordered ones....
Shouldn't it be the opposite?

zachleat commented

@TigersWay hmm—yeah, that makes sense. This will require a major version bump.

dixonge commented

@TigersWay hmm—yeah, that makes sense. This will require a major version bump.

Any timeframe on this? I'm still finding things sorting extremely randomly, and am unwilling to specify order for 100+ nav items. Guess I'm probably pushing the limits of the concept of a 'nav menu' and should probably just use collections.all as an index or something... :(

aaronstezycki commented

Is there anyway to order items alphabetically by default?

TigersWay commented

All troubles are coming from here:

eleventy-navigation/eleventy-navigation.js

Line 18 in c05ff04

return (a.order || 0) - (b.order || 0);

  • Only numeric ordering
  • 0 is default (unordered first)

I could prepare a PR... (?)