ITDP / the-online-brt-planning-guide

Online collaborative version of the BRT Planning Guide
https://brtguide.itdp.org
Other
6 stars 20 forks source link

Next button #189

Closed aszasz closed 7 years ago

aszasz commented 7 years ago

This includes previous and next button with same name/number and style seen in the content.

It is still to properly css-styled and have the content (text of the button) discussed before merge. (not metioning a review from @jonasmalacofilho to request addition of best practices of programming that may have escaped).

aszasz commented 7 years ago

I think it is good for merge, because is a useful feature. We should eventually open issues about:

jonasmalacofilho commented 7 years ago

I think this looks great and I will merge it as is!

In the future, I think we should:


An example on why I think the "previous" button is confusing: when I've just entered a new chapter, the "previous" button will show me the last section of the previous chapter (as it should, since it's the opposite of "next"). However, that's doesn't seem useful at all: what I would perhaps like to be reminded of is the name of the last volume or chapter (depending on whether I've entered a new volume or not), something we already have on the nav bar.