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

fix drawer scroll through #98

Merged
merged 4 commits into from
Dec 28, 2021
Merged

Conversation

caoML
Copy link
Contributor

@caoML caoML commented Dec 25, 2021

sub modules pr address: Tencent/tdesign-common#59

其中涉及到了src/drawer/props.ts、src/drawer/type.ts这两个文件的变更: 增加preventScrollThrough属性

@github-actions
Copy link
Contributor

github-actions bot commented Dec 25, 2021

PR 预览产物在 https://preview-pr98-tdesign-vue.surge.sh

@caoML caoML changed the title fix scroll through fix drawer scroll through Dec 25, 2021
@caoML caoML closed this Dec 26, 2021
@@ -73,6 +77,9 @@ export default mixins(ActionMixin, getConfigReceiverMixins<Vue, DrawerConfig>('d
justifyContent: this.placement === 'right' ? 'flex-start' : 'flex-end',
};
},
lockFullscreen(): boolean {
return this.preventScrollThrough;
},
Copy link
Collaborator

Choose a reason for hiding this comment

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

preventScrollThrough 已经是独立的 boolean 变量,不需要再定义 lockFullscreen

import { ClassName, Styles } from '../common';
import ActionMixin from '../dialog/actions';

type FooterButtonType = 'confirm' | 'cancel';

const name = `${prefix}-drawer`;
const lockClass = `${prefix}-drawer-lock`;
Copy link
Collaborator

Choose a reason for hiding this comment

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

  • t-drawer-lock 更名为 t-drawer--lock
  • 同步修改 t-loading-lockt-loading--lock
  • t-dialog--lock(Dialog 也使用同样的方式处理滚动穿透)

@caoML caoML reopened this Dec 26, 2021
@chaishi chaishi merged commit a0a62e4 into Tencent:develop Dec 28, 2021
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.

当抽屉打开时,body有滚动条时可以滚动
2 participants