Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(taro-components): 修复 picker date,start end 无效 #2003

Merged
merged 2 commits into from
Jan 24, 2019

Conversation

chenxtdo
Copy link
Contributor

No description provided.

@luckyadam
Copy link
Member

@jinjinjin0731 三金哥帮忙 review

@sweet-smail
Copy link

版本:image
代码:
image
错误信息:
image

@jinjinjin0731
Copy link
Contributor

有点问题,正在修复 当 start='2018-10-20' end='2019-1-24' 这个的时候
1, 滑动 2018 的 月份自动变成了 1月。
2, 当 end 设置为为当天日期,会出现时间校验的错误

@chenxtdo
Copy link
Contributor Author

@jinjinjin0731 你好,我有提交一次 Pull requests ,可能解决了当前设置 start end 的问题,可以 review 下

@jinjinjin0731
Copy link
Contributor

jinjinjin0731 commented Jan 23, 2019

@chenxtdo 对的 我说的就是你的 PR 代码。 你可以按我上面说的那么操作,可以复现问题,我在基于你的 PR 修复

@chenxtdo
Copy link
Contributor Author

@jinjinjin0731 我试了下,并没有复现,请看下下面的演示

image

qq20190123-111826-hd 2019-01-23 11_20_08

@jinjinjin0731
Copy link
Contributor

jinjinjin0731 commented Jan 23, 2019

@chenxtdo value 去掉 仅设置 start 与 end

@chenxtdo
Copy link
Contributor Author

@jinjinjin0731 我使用的是 1.2.8 版本,请问@tarojs/components引用的是不是 dist/index.js ?
我是通过原项目 taro build后获取dist/index.js替换我项目里的,重行运行做的测试

image

value 不设置不是取的是今天?

image

@jinjinjin0731
Copy link
Contributor

@chenxtdo 取得是今天,但是有个全局变量没有被更新。 在 updateDay 的时候又取回当前时间了( 初步定位,还在改

@sweet-smail
Copy link

@chenxtdo
image
渲染结果:
image
这个很明显不正确啊 麻烦你贴一下你的版本号

@chenxtdo
Copy link
Contributor Author

@Pecking 同学,我测试的是我改过后的代码,并不是1.2.8的版本。你可以等 @jinjinjin0731 修复,并更新。😂

@jinjinjin0731 jinjinjin0731 merged commit e624b11 into NervJS:master Jan 24, 2019
@jinjinjin0731
Copy link
Contributor

thx

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.

4 participants