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

feat: 模态弹窗添加最大化和退出最大化功能 #320

Merged
merged 2 commits into from
Aug 8, 2023

Conversation

xuxzh
Copy link
Contributor

@xuxzh xuxzh commented Jul 17, 2023

模态弹窗添加最大化和退出最大化功能,同时修改说明文档并新增对应demo

-webkit-transition: none;
transition: none;
transform: none;
width: 100vw !important;
Copy link
Collaborator

Choose a reason for hiding this comment

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

image
最大化后水平和竖直方向会有滚动条存在

Copy link
Contributor Author

Choose a reason for hiding this comment

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

已优化,顺便优化了mobile的样式。

  maximize() {
    this.maximized = !this.maximized;
    if (this.maximized) {
      this.width = '100vw';
      this.renderer.setStyle(this.document.body, 'overflow', 'hidden');
    } else {
      this.width = this._oldWidth;
      this.renderer.setStyle(this.document.body, 'overflow', 'auto');
    }
  }

Copy link
Collaborator

Choose a reason for hiding this comment

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

Good!

Copy link
Collaborator

Choose a reason for hiding this comment

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

This branch is out-of-date with the base branch,我们不久前发布了16.0.0版本,请拉取最新代码,然后我们合入

Copy link
Contributor Author

Choose a reason for hiding this comment

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

你好,已拉取最新代码。之前没操作过,不知道方式对不对。

@wangyaju wangyaju merged commit 8456cba into DevCloudFE:master Aug 8, 2023
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.

3 participants