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

1、问题/原因: #963

Merged
merged 2 commits into from
Jun 10, 2022
Merged

1、问题/原因: #963

merged 2 commits into from
Jun 10, 2022

Conversation

huoyuhao
Copy link
Contributor

@huoyuhao huoyuhao commented Jun 9, 2022

1、问题/原因:
当前Dialog 是通过transform 做动画和位移的,transform 属性会导致浏览器创建一个 layer 层从而使内部子节点的 fixed 属性定位失效

调整项目:
使用flex布局解决之前居中布局使用transform位移导致子节点的 fixed 失效

2、问题/原因:
没有适配dialog长度超出屏幕,使用最大高度100vh,然后内部body进行滚动,不支持mask滚动

调整项目:
新增wrap、position两层div,用于dialog内容过长出现滚动场景

3、问题原因:
使用新的适配方案,调整拖拽实现

调整项目:
优化拖拽,重写定位位置计算,适配top定位
当dialog内容超出屏幕,不能进行拖拽

当前Dialog 是通过transform 做动画和位移的,transform 属性会导致浏览器创建一个 layer 层从而使内部子节点的 fixed 属性定位失效

调整项目:
使用flex布局解决之前居中布局使用transform位移导致子节点的 fixed 失效

2、问题/原因:
没有适配dialog长度超出屏幕,使用最大高度100vh,然后内部body进行滚动,不支持mask滚动

调整项目:
新增wrap、position两层div,用于dialog内容过长出现滚动场景

3、问题原因:
使用新的适配方案,调整拖拽实现

调整项目:
优化拖拽,重写定位位置计算,适配top定位
当dialog内容超出屏幕,不能进行拖拽
@github-actions
Copy link
Contributor

github-actions bot commented Jun 9, 2022

完成

@honkinglin honkinglin merged commit 4b144cc into Tencent:develop Jun 10, 2022
@github-actions github-actions bot mentioned this pull request Jun 13, 2022
4 tasks
@huoyuhao huoyuhao deleted the feat/dialog/position branch July 27, 2022 08:48
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.

None yet

3 participants