pagination icon indicating copy to clipboard operation
pagination copied to clipboard

Published 20 hours ago • verified publisher icon react-component

Add pagerCount props

Open liuchuzhang opened this issue 6 years ago • 27 comments

ant-design/ant-design#14683

liuchuzhang avatar Feb 22 '19 06:02 liuchuzhang

Deploy preview for rc-pagination failed.

Built with commit 9a55e81914a73e07f28f4b02acf058e5bfaebbfc

https://app.netlify.com/sites/rc-pagination/deploys/5c809ff6d1d694000831e739

ant-design-bot avatar Feb 22 '19 06:02 ant-design-bot

Coverage Status

Coverage increased (+0.9%) to 90.119% when pulling 9a55e81914a73e07f28f4b02acf058e5bfaebbfc on liuchuzhang:jump into 4b1c6b2c190ef15b84294c13224d4f36e17f0de4 on react-component:master.

coveralls avatar Feb 22 '19 06:02 coveralls

只做了隐藏，应该再添加设置最大分页跨度来约束最大可跨范围来达到 https://github.com/ant-design/ant-design/issues/14683#issuecomment-463515852 的想法

yoyo837 avatar Feb 25 '19 10:02 yoyo837

@yoyo837 好的。

liuchuzhang avatar Feb 26 '19 02:02 liuchuzhang

@afc163 这个向后 5 页的 title 该怎么弄😄

liuchuzhang avatar Feb 26 '19 03:02 liuchuzhang

@yoyo837 像这种语言也不知道用不用阿拉伯数字 我看 element ui 没有 title 是要牺牲这个 title 还是有别的语言方案解决?

image image image

liuchuzhang avatar Feb 26 '19 06:02 liuchuzhang

你准备加什么

yoyo837 avatar Feb 26 '19 07:02 yoyo837

@yoyo837 你好，我是说这个把这个按钮数量设置为 10 的话, 向前 5 页将是向前 10 页。title 也要随之改变。 title 是鼠标 hover 在这里的 title。 image

liuchuzhang avatar Feb 26 '19 07:02 liuchuzhang

我是说这个把这个按钮数量设置为 10 的话, 向前 5 页将是向前 10 页

这个按钮数量设置为 10 也保持向前向后 5 页吧，就是两者不关联。

afc163 avatar Feb 26 '19 07:02 afc163

可以一直保持向前x页、向后x页吧。另外添加国际化的话一般做法是不知道怎么翻译就保留国际语言英语。

yoyo837 avatar Feb 26 '19 07:02 yoyo837

@afc163 这样子吗，我感觉要有 10 个按钮的话向前 10 页更好啊😄

liuchuzhang avatar Feb 26 '19 07:02 liuchuzhang

我感觉有 pagerCount 就够了，剩下的改进一下 itemRender 可以做出来这个效果。不一定要额外的 hideBonudary。

zombieJ avatar Feb 26 '19 12:02 zombieJ

@zombieJ 请问现在是不要 hideBonudary ，同时按钮数量设置为 10 也保持向前向后 5 页。 另外我看现有的 itemRender 是在按钮的内部进行自定义，我们是要改进 itemRender 自定义整个按钮吗？

liuchuzhang avatar Feb 27 '19 01:02 liuchuzhang

@zombieJ 请问现在是不要 hideBonudary ，同时按钮数量设置为 10 也保持向前向后 5 页。 另外我看现有的 itemRender 是在按钮的内部进行自定义，我们是要改进 itemRender 自定义整个按钮吗？

itemRender 返回 null 时就不渲染这个 Pager，因为我们的按钮是: Left First Jumper Pages Jumper Last Right

按照需求，我们只要改成： Left Pages Right

然后改写 Left 和 Right 的逻辑从 1 页改成 n 页就行了。

zombieJ avatar Feb 27 '19 02:02 zombieJ

@zombieJ 抱歉昨天有点忙，请 review 一下。 例子在 pagerCount.html

liuchuzhang avatar Feb 28 '19 01:02 liuchuzhang

然后改写 Left 和 Right 的逻辑从 1 页改成 n 页就行了。

这个就没必要了因为 ant-design/ant-design#14683 他说的是类似百度那种，百度的 Left 和 Right 是只跳一页的。

liuchuzhang avatar Feb 28 '19 02:02 liuchuzhang

CI broken...

zombieJ avatar Feb 28 '19 02:02 zombieJ

咋回事 我只删掉了一行文档

liuchuzhang avatar Feb 28 '19 02:02 liuchuzhang

本地跑看看能过不？

zombieJ avatar Feb 28 '19 02:02 zombieJ

过了一次, 然后又过不了了, 我好像发现了 Bug 我下午弄一下

liuchuzhang avatar Feb 28 '19 04:02 liuchuzhang

加些 test case 来测一下新的计算方式和以前在生成 pager 的数量上是否相同。多搞一些边界值测测~

zombieJ avatar Mar 01 '19 03:03 zombieJ

@zombieJ pagerCount 需要大于 3 是否要在文档中体现。

liuchuzhang avatar Mar 01 '19 06:03 liuchuzhang

不能小于 3，小于 3 不好翻页了。

liuchuzhang avatar Mar 01 '19 06:03 liuchuzhang

不能小于 3，小于 3 不好翻页了。

还有前一页/后一页按钮可以点~

zombieJ avatar Mar 01 '19 06:03 zombieJ

@zombieJ

liuchuzhang avatar Mar 04 '19 09:03 liuchuzhang

加些 test case 来测一下新的计算方式和以前在生成 pager 的数量上是否相同。多搞一些边界值测测~

zombieJ avatar Mar 05 '19 07:03 zombieJ

冲突了

afc163 avatar Mar 13 '20 07:03 afc163