finnsson / pagerjs

The Single Page Application Framework for KnockoutJS
http://pagerjs.com/
MIT License
271 stars 68 forks source link

Page.path(string) not documented #161

Open rafal-brize opened 10 years ago

rafal-brize commented 10 years ago

It is not documented that i can construct a relative path of a subpage with path(string).

For example: Let the current page to be #/first

pager.activePage$().path('alert')

to construct a #/first/alert

Is this a proper way to do this in javascript to navigate to a subpage? pager.navigate(pager.activePage$().path('alert'))

If so, why it is not documented?

finnsson commented 10 years ago

It seems like I got some copy-n-paste bug in the documentation. The page Path should really document this feature but is instead documenting the find-method. I'll fix it. Thanks for the heads up.

Regarding your second question I would say it depends on your current context. Sometimes your solution is the best, sometimes you might be able to navigate in some other way. Internally (in pager.js) I'm usually doing something similar to your code example.