-
Notifications
You must be signed in to change notification settings - Fork 308
fix(popup): deal with container height overflow #3586
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
Conversation
commit: |
Size Change: +1 B (0%) Total Size: 187 kB
ℹ️ View Unchanged
|
@betavs hello~ 有复现的录屏或截图吗?我未能复现😂。在我的真机和模拟器上,pr 前后效果是一样的,我看不出区别 😅 |
@anlyyao 从你目前提供的图片可以看出滚动到最底部了,但是内容是没有展示完整的。 <view style="width: 300px; height: 100vh; overflow: auto"> 在该PR中需要设置 |
@betavs 实际上,上一张截图并没有滚动到最底部。pr前后,在我的真机(13 pro) 上,height 无论是 100vh 还是 100% 都无法滚动。 |
@anlyyao 在模拟器中是可以滚动,真机用下面的容器来包裹测试吧 <scroll-view scroll-y style="width: 300px; height: 100%; overflow: auto"> |
那直接 |
631b687
to
4e56b01
Compare
🤔 这个 PR 的性质是?
🔗 相关 Issue
fix #3569
💡 需求背景和解决方案
📝 更新日志
fix(Popup): 当
placement
为'left' | 'right'
配合usingCustomNavbar
时,容器高度溢出的问题本条 PR 不需要纳入 Changelog
☑️ 请求合并前的自查清单