Skip to content

Conversation

qiaolb
Copy link

@qiaolb qiaolb commented Mar 15, 2017

增加了属性:
value: 如果设置将会覆盖percent, 如果不设置就是要percent
minValue: 进度的起点值,例如温度仪表盘,起点不一定是0,这个值缺省0;
maxValue:进度的终点值,缺省100
openWidth:半圆开口的宽度,缺省0
beginPosition:进度起始位置,更重要的是影响半圆开口方向,目前设置了上下左右四个方向,值为:'top', 'bottom', 'left', 'right',缺省向上,保持和以前相同。

这部分修改不影响之前的功能。

@afc163
Copy link
Member

afc163 commented Mar 15, 2017

增加了太多属性了。

  • gapWidth
  • gapPosition

应该就足够了。

@qiaolb
Copy link
Author

qiaolb commented Mar 15, 2017

@afc163 也可以,让使用者自己将值转换为百分比。我修改一下

@qiaolb
Copy link
Author

qiaolb commented Mar 15, 2017

@afc163 已经删除了remove value, minValue, MaxValue,只保留了gapWidth和gapPosition

@afc163
Copy link
Member

afc163 commented Mar 15, 2017

@ddcat1115

@qiaolb
Copy link
Author

qiaolb commented Mar 15, 2017

ci的环境好像不支持ES6的一些写法,检查失败了。

@afc163
Copy link
Member

afc163 commented Mar 15, 2017

- 4.0.0

改为 6 即可。

@coveralls
Copy link

Coverage Status

Coverage decreased (-15.2%) to 75.676% when pulling ebbd741 on qiaolb:master into 725f238 on react-component:master.

@afc163
Copy link
Member

afc163 commented Mar 16, 2017

补个用例?

@coveralls
Copy link

Coverage Status

Coverage increased (+2.3%) to 93.243% when pulling 467a17a on qiaolb:master into 725f238 on react-component:master.

<td>gapWidth</td>
<td>Number</td>
<td>0</td>
<td>the gap width of half circle</td>
Copy link
Member

Choose a reason for hiding this comment

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

其实这个我是有点困惑的,是按百分比还是绝对值?绝对值的话,宽度又是相对于什么?

Copy link
Author

@qiaolb qiaolb Mar 16, 2017

Choose a reason for hiding this comment

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

百分比的确解释起来比较麻烦,绝对值应该比较直观

HISTORY.md Outdated
---

## 2.1.0
- Add `value` `minValue` `maxValue` `openWidth` `beginPosition` props.
Copy link
Member

Choose a reason for hiding this comment

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

Change log 和实际修改不同步。

Copy link
Author

Choose a reason for hiding this comment

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

Sorry,我修改一下

@benjycui
Copy link
Member

👍

@coveralls
Copy link

Coverage Status

Coverage increased (+2.3%) to 93.243% when pulling 95b2756 on qiaolb:master into 725f238 on react-component:master.

@afc163 afc163 merged commit 3549bf3 into react-component:master Mar 16, 2017
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