Nah, that seems fine. You might want to provide a few example URLs on the
`pageUrl` attr though, just to demonstrate how the replacements work.


On Fri, Apr 12, 2013 at 1:01 PM, Anthony Pipkin <notifications@github.com>wrote:

> In src/paginator/js/paginator-url.js:
>
> > +     @method nextPageUrl
> > +     @return String | NULL
> > +     */
> > +    nextPageUrl: function () {
> > +        return (this.hasNextPage() && this.formatPageUrl(this.get('page') + 1)) || null;
> > +    },
> > +
> > +    /**
> > +     Returns a formated URL for the provided page number.
> > +     @method formatPageUrl
> > +     @return String | NULL
> > +     */
> > +    formatPageUrl: function (page) {
> > +        var url = this.get('url');
> > +        if (url) {
> > +            return Y.Lang.sub(url, {
>
> This is now commented on the pageUrl ATTR. Does this need to be noted here
> as well?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/yui/yui3/pull/466/files#r3779122>
> .
>