-
Notifications
You must be signed in to change notification settings - Fork 363
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(menu): expandType=popup时箭头方向错误问题 #806
Conversation
fc4e37b
to
f97b101
Compare
这个好像是设计如此.. |
真的吗。。感觉不符合直觉,popup时是浮在右边的哦 |
让设计师看看 @Wen1kang |
e262ec1
to
5ce805a
Compare
Codecov Report
@@ Coverage Diff @@
## develop #806 +/- ##
===========================================
+ Coverage 57.56% 61.60% +4.04%
===========================================
Files 346 354 +8
Lines 12828 13260 +432
Branches 2405 2499 +94
===========================================
+ Hits 7384 8169 +785
+ Misses 5200 4837 -363
- Partials 244 254 +10
Continue to review full report at Codecov.
|
的确这里是有问题,设计已修复了,五一节后我们修改menu和dropdown的样式 @fengxianqi |
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.
LGTM
🤔 这个 PR 的性质是?
🔗 相关 Issue
Tencent/tdesign-vue-next#644
💡 需求背景和解决方案
修复前:https://stackblitz.com/edit/angular-kqnx7y?file=src%2Fdemo.vue
![image](https://user-images.githubusercontent.com/15716381/165099928-7ba89922-33cf-4766-91aa-a4e16aa27fe7.png)
修复后:
![image](https://user-images.githubusercontent.com/15716381/165100410-2fee7fd9-9177-4fda-a4ef-93c409ece399.png)
📝 更新日志
fix(menu): expandType=popup时箭头方向错误问题
fix(组件名称): 处理问题或特性描述 ...
本条 PR 不需要纳入 Changelog
☑️ 请求合并前的自查清单