Skip to content

Conversation

fireairforce
Copy link
Contributor

relative issue: ant-design/ant-design#24937

@fireairforce
Copy link
Contributor Author

这里实际上是 monthquarter 这边的参数没传,所以这两个这里就不会渲染写好的伪类,而是使用默认的那个。

@fireairforce
Copy link
Contributor Author

这里与之类似的是 year 的那个picker,那边伪类有生效。

@codecov
Copy link

codecov bot commented Jun 16, 2020

Codecov Report

Merging #86 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #86   +/-   ##
=======================================
  Coverage   99.55%   99.55%           
=======================================
  Files          42       42           
  Lines        1792     1792           
  Branches      529      529           
=======================================
  Hits         1784     1784           
  Misses          6        6           
  Partials        2        2           
Impacted Files Coverage Δ
src/panels/MonthPanel/MonthHeader.tsx 100.00% <ø> (ø)
src/panels/QuarterPanel/QuarterHeader.tsx 100.00% <ø> (ø)
src/panels/YearPanel/YearHeader.tsx 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4320d11...213c157. Read the comment docs.

@fireairforce
Copy link
Contributor Author

image

image

上面两张图是这个button这里的比较图。

@fireairforce
Copy link
Contributor Author

image

顺便问一下这三个picker里面的这个key是不是应该不相同= =

Copy link
Member

@xrkffgg xrkffgg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

嗯, key 改一下

@fireairforce
Copy link
Contributor Author

ok,done了

@xrkffgg xrkffgg changed the title feat: add icons on month and quarter fix: icons in month and quarter Jun 16, 2020
@xrkffgg xrkffgg requested a review from zombieJ June 16, 2020 07:17
@zombieJ
Copy link
Member

zombieJ commented Jun 16, 2020

我想不起来为什么要写 key 了,看起来这里的 key 是可以删掉的?没用就删了吧。

@fireairforce
Copy link
Contributor Author

已删~

@zombieJ
Copy link
Member

zombieJ commented Jun 16, 2020

Year 那个是不是也可以删了?其他地方有用到这个没用的 key 都可以顺道清理一下哈~

@fireairforce
Copy link
Contributor Author

fireairforce commented Jun 16, 2020

ok,刚才删了,忘了push上来。。。。已经都删了

@zombieJ zombieJ merged commit ea972dc into react-component:master Jun 16, 2020
@zombieJ
Copy link
Member

zombieJ commented Jun 16, 2020

+ rc-picker@1.7.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants