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

spinbox使用反馈~~~~ #41

Open
sirzxj opened this issue Sep 24, 2012 · 2 comments
Open

spinbox使用反馈~~~~ #41

sirzxj opened this issue Sep 24, 2012 · 2 comments

Comments

@sirzxj
Copy link
Contributor

sirzxj commented Sep 24, 2012

对ie的支持好像不是很好~

  • 查看了下源码,发现第65行的代码 ,发现这里木有creat的元素span没有闭合标签,chrome下是没事的,但是在ie下面就会报错了,这个元 素是无法create的

$containerEl = $(D.create('<span class="'+ getCls.container + '">'))

  • 还有一处是内部的两个函数_eventOnValide和_eventOnValide都有用到字符串的trim方法,但是这个方法ie 浏览器好像也不原生支持的。

交互方面的

左右的增减按钮的样式最好能够随着交互的状态而变动(这个是我这边日常上的需求)显示出三种状态:normal,hover,disabled,见下图

img

数字增减器,好像仅仅是金额

源码中会有 toFixed(2),每次竖直都有两位小数,这个组件不仅仅可以用于金额方面,比如说detail那边的购买宝贝个数,就是整数吖,所以,让用户自己去做主吧~

@flyinhigh
Copy link
Contributor

感谢反馈,我会在后续版本中改进~

@flyinhigh
Copy link
Contributor

已经fix掉了,等待下个版本发布。

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

No branches or pull requests

2 participants