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: 修复了内容区没有居中于弹窗的问题 #155

Closed
wants to merge 0 commits into from
Closed

fix: 修复了内容区没有居中于弹窗的问题 #155

wants to merge 0 commits into from

Conversation

Qujixiang
Copy link

修复了无线网络、有线网络(添加网络配置)、DSL(创建PPPoE连接)、VPN(添加VPN)一共四个页面的内容区没有居中于弹窗的问题。

修复的具体内容:

  1. 对内容区的布局的左侧content margin进行了修改,使得内容区居中了。
  2. 对“添加VPN”页面的“VPN类型”控件的左侧content margin进行了修改,使得该行居中了。

Issue: #141

Log: 修复了内容区没有居中于弹窗的问题

修改前的页面:
WC_85K@H)O)A1(LCYU3HBHC

修改后的页面:
1Z(O9WK{68U0~W8KC_$1P$X

@github-actions
Copy link

github-actions bot commented Apr 18, 2023

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@Qujixiang
Copy link
Author

recheck

@Decodetalkers
Copy link
Contributor

你代码里面设置了margin左边是24,右边是8,然后产生结果是居中,那么问题不是出在这里。这种写法明显不是居中的意思。你是偶然让它达到居中效果了

@Qujixiang
Copy link
Author

那我应该怎么改呢?是修改布局吗?我分析了一下造成没有居中的原因是因为布局右侧比左边多了一个滚动条的宽度,所以我应该把滚动条的宽度加到左侧吗?

@Decodetalkers
Copy link
Contributor

把滚动条隐藏试试

@Qujixiang
Copy link
Author

好的

@Qujixiang
Copy link
Author

你好,问题已经被我同学解决了,这个PR可以由我同学来提交吗?我关闭这个PR可以吗?

@Qujixiang Qujixiang closed this Apr 21, 2023
@Decodetalkers
Copy link
Contributor

可以

@Qujixiang
Copy link
Author

可以
同学的新的PR已经提交了,一共修复了两个bug,一个是内容区对齐,一个是导出和保存按钮宽度一致性的问题,地址:

麻烦您过目一下,谢谢!😀(如果合适的话可以尽快merge嘛)

@Decodetalkers
Copy link
Contributor

这俩好像都没有正确解决,还挺棘手的。。可能都不是这样修

@Qujixiang
Copy link
Author

这俩好像都没有正确解决,还挺棘手的。。可能都不是这样修

请问一下,
内容区居中是不是同时要保留滚动条?
button与弹窗间距一致这个是哪里不对呢?

@Decodetalkers
Copy link
Contributor

这俩好像都没有正确解决,还挺棘手的。。可能都不是这样修

请问一下, 内容区居中是不是同时要保留滚动条? button与弹窗间距一致这个是哪里不对呢?

我不知道。还要看。。还不知道问题出在哪里

@Qujixiang
Copy link
Author

这俩好像都没有正确解决,还挺棘手的。。可能都不是这样修

请问一下, 内容区居中是不是同时要保留滚动条? button与弹窗间距一致这个是哪里不对呢?

我不知道。还要看。。还不知道问题出在哪里

第一个内容区居中的问题解决了,保留了滚动条,问题出在没有给布局控件设置横向对齐。请您查看下。

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.

2 participants