-
Notifications
You must be signed in to change notification settings - Fork 280
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
Docs/progress 修复首次进入时与设置的stroke-width的延迟显示和兼容性问题 #3133
Conversation
@huxinhai 这个场景 方便提供录屏👀👀吗 |
src/progress/progress.wxml
Outdated
@@ -80,7 +80,7 @@ | |||
> | |||
<view | |||
class="{{classPrefix}}__canvas--inner {{prefix}}-class-bar" | |||
style="{{innerDiameter? 'width:'+ innerDiameter*2 + 'rpx;' + 'height:'+ innerDiameter*2 + 'rpx;': ''}}" | |||
style="{{strokeCircleWidth? '--td-progress-stroke-circle-width:'+ strokeCircleWidth : ''}}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
style="{{heightBar? '--td-progress-stroke-circle-width:' + heightBar + 'px': ''}}"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
getInnerDiameter() 可以整段移除了
改完之后要更新快照文件 |
/update-snapshot |
1 similar comment
/update-snapshot |
Duplicate of #3139 |
🤔 这个 PR 的性质是?
🔗 相关 Issue
💡 需求背景和解决方案
背景: 在使用progress组件时 当小程序首次进来的时候 会出现外部设置了stroke-width 但是在最开始的时候还是能看到组件本身的默认值 要等一小会才会更新视图 虽然说理论上都要先做单位转换, 直接*2 除了375的屏幕,其他屏幕下会有出入 #但是我觉得起码这样可以先解决大问题😄😄
📝 更新日志
fix(Progress): 修复环形进度条首次加载时,
strokeWidth
线宽延迟显示的问题本条 PR 不需要纳入 Changelog
☑️ 请求合并前的自查清单