http2-explained icon indicating copy to clipboard operation
http2-explained copied to clipboard

Name parts with number less 10 with a leading zero

Open olegstepura opened this issue 10 years ago • 9 comments

Now github orders files the wrong way. I suggest solving this by adding a leading zero to numbers < 10

Before:

part1.md
part10.md
part11.md
part12.md
part13.md
part14.md
part2.md
part3.md
part4.md
part5.md
part6.md
part7.md
part8.md
part9.md

After

part01.md
part02.md
part03.md
part04.md
part05.md
part06.md
part07.md
part08.md
part09.md
part10.md
part11.md
part12.md
part13.md
part14.md

olegstepura avatar Dec 19 '15 17:12 olegstepura

Right, but does it really matter in which order github shows them?

bagder avatar Dec 19 '15 17:12 bagder

I was reading it on github. Ordering did matter. After part 9 I had to scroll back to top to find part 10. I could think there is no part 10 at all. If there would be a link #98, it would be even better.

olegstepura avatar Dec 19 '15 17:12 olegstepura

Oh, you were reading it on github... Can I ask you why you read it here? The book renders much better on the actual book site.

bagder avatar Dec 19 '15 17:12 bagder

I found it here and read it here =)

olegstepura avatar Dec 19 '15 17:12 olegstepura

Try reading the book at http://http2-explained.haxx.se/ instead =)

bagder avatar Dec 19 '15 17:12 bagder

BTW github renders thing very well, so reading it on github is ok. Maybe I'm the only one who did it this way, but I only suggest after reading all book and having some experience. Others may be just lazy enough to suggest such things ;)

olegstepura avatar Dec 19 '15 17:12 olegstepura

Oh sure, I appreciate the feedback, I don't mean to imply anything else. It's just that a kind of "next chapter" link makes less sense in a PDF version or in the html version as it is rendered on http2-explained.haxx.se so I'm not sure how to do it to make it nice all over.

bagder avatar Dec 19 '15 17:12 bagder

Well, it's up to you, actually. I cannot insist. Tha't why I opened an issue, not a pull request: to discuss this option.

olegstepura avatar Dec 19 '15 17:12 olegstepura

Ok, just close the #98 issue as won't fix ;)

olegstepura avatar Dec 19 '15 17:12 olegstepura