urbit / urbit.org

The source for urbit.org
https://urbit.org
MIT License
93 stars 197 forks source link

Markdown YAML order sorting incorrectly. #845

Closed xykj61 closed 3 years ago

xykj61 commented 7 years ago

The reason the docs/arvo pages seem out of order (Security Drivers and API's coming before basic Hoon material) is because the YAML sorting is not functioning properly. Tree is sorting 11 and 12 before 2 and 3. I tried fixing this by changing pages 0-9 to 00-09. This didn't work, but changing the page range from 1-12 to 11-22 fixed this.

Going to PR this so it works, but not sure if there's a change that can be made to make 1-12 work (just to make to make it normal, as I imagine this is going to be pretty annoying for people in the future wanting to, say, host their own blogs or static pages through Tree).

cgyarvin commented 7 years ago

The numbering model makes it a huge pain to add a chapter in the middle of something. -1. Would be nice to do it some other way.

On Wed, Feb 8, 2017 at 11:17 AM, Keaton Dunsford notifications@github.com wrote:

The reason the docs/arvo pages seem out of order (Security Drivers and API's coming before basic Hoon material) is because the YAML sorting is not functioning properly. Tree is sorting 11 and 12 before 2 and 3. I tried fixing this by changing pages 0-9 to 00-09. This didn't work, but changing the page range from 1-12 to 11-22 fixed this.

Going to PR this so it works, but not sure if there's a change that can be made to make 1-12 work (just to make to make it normal, as I imagine this is going to be pretty annoying for people in the future wanting to, say, host their own blogs or static pages through Tree).

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/urbit/docs/issues/129, or mute the thread https://github.com/notifications/unsubscribe-auth/AALyATURtUd7f4FFeZknsMYi3PScpfzCks5rahTRgaJpZM4L7PVg .

galenwp commented 7 years ago

This is a tree bug, and should be fixed there. Can you move this issue there? I understand the number ordering is annoying, but we're just matching Jekyll here. Happy to hear another solution, but fixing the existing intended behavior isn't that hard.

xykj61 commented 6 years ago

This should be fixed in new static Tree, so closing this. @ixv, can you confirm?