search

Danielss89 / ZfcUserAdmin

An admin interface for ZfcUser
36 stars 45 forks source link

Add option for number of users per page (pagination) #57

Open fabwu opened 10 years ago

stijnhau commented 10 years ago

Seems like a good welcome change. Except the view keep telling next 10 shoudl be the value from the moduleOption class.

fabwu commented 10 years ago

I believe this button means the next 10 pages and not the next 10 items...

In my project I replace this button with a link to the next page. Do you want this?

stijnhau commented 10 years ago

No realy means next 10 items(users).

stijnhau commented 10 years ago

Can you also change it in the view?

fabwu commented 10 years ago

As you can see in the comment this link mean the next 10 pages. But you can create a seperate PR with your feature.

<!-- Previous page link -->
<li<?php echo (isset($this->previous)) ? '' : ' class="disabled"'; ?>>
    <a href="<?php echo $this->url('zfcadmin/zfcuseradmin/list', array('p' => $this->previous-9)); ?>">&lt; Previous 10</a>
</li>
fabwu commented 10 years ago

@Danielss89 Can you clarify the meaning of this link, please?

stijnhau commented 9 years ago

@Danielss89 Any comment?