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

Support frameless window and custom css #1172

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

JonasGao
Copy link

@JonasGao JonasGao commented Mar 4, 2024

  1. 支持 Frameless Window

image

image

  1. 支持自定义样式

image

@qishibo
Copy link
Owner

qishibo commented Mar 17, 2024

title bar设置成自定义是为了扩大可视化窗口么?
还有自定义样式的需求貌似不是太多,你这里主要是自定义了哪些地方,可以截个图看看么

@JonasGao
Copy link
Author

title bar设置成自定义是为了扩大可视化窗口么?

是的。用来扩大。不过我做的 Frameless UI 很简单。不知道行不行。可能还需要再加个开关用来开启关闭这个功能。

还有自定义样式的需求貌似不是太多,你这里主要是自定义了哪些地方,可以截个图看看么

好。前边几天都比较忙,忘记截图了。我尽快截图上来。

@qishibo
Copy link
Owner

qishibo commented Mar 19, 2024

是的。用来扩大。不过我做的 Frameless UI 很简单。不知道行不行。可能还需要再加个开关用来开启关闭这个功能。

是的不过感觉去掉了原生标题栏感觉有些突兀,可以再斟酌一下

好。前边几天都比较忙,忘记截图了。我尽快截图上来。

我是指你自定义css文件想要改样式的情况,因为整个APP样式基本都是定制化的,定制的话需要重写的css也是相当多的,正式版本没有控制台窗口权限,想要拿到元素路径也不太容易,所以想看看你到底自定义了哪里,或者把css贴上来也行

@JonasGao
Copy link
Author

不好意思,之前的样式找不到了。😅 因为后来自己做了 Frameless 之后布局变了,需要调的我也直接写到PR里了。所以那个样式文件忘了放哪了。

当时调的地方主要就是觉得左边的菜单在有颜色条的时候,跟没有颜色的 Item 边框不齐有点难受。

PR里涉及样式微调的就这俩

22bdd46#diff-2b91248f1950ba4e4e249df08b2785f491872d8629cc58a76c7013f538ca6ff9R245

4ffb702#diff-871044680db54942b78e1b8efee6053b25e3a9c20e1b9523a4b4ef67a887e651R345

@qishibo
Copy link
Owner

qishibo commented Mar 21, 2024

感觉你有点electron的经验

所以你为了调整边距就写了个自定义css的功能?😂杀鸡牛刀了啊
不过自定义css的功能感觉确实需求不大,太小众了,可以暂时去掉

自定义title bar那个还有点问题,不知道为啥最大化窗口时,toggle那个方法不好使了,我是mac,你什么系统,toggle的时候能恢复最大化之前的窗口么

@@ -346,7 +346,7 @@ export default {

.connection-menu .connection-name {
margin-right: 115px;
padding-right: 6px;
Copy link
Owner

Choose a reason for hiding this comment

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

这里确实,right没啥用,应该用left,这样左边就不那么贴边了。
我都感觉这里当时手滑敲错了

@@ -242,7 +242,7 @@ export default {
/*menu ul*/
.connection-menu {
margin-bottom: 8px;
padding-right: 6px;
Copy link
Owner

Choose a reason for hiding this comment

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

这个没必要,可以删掉

@JonasGao
Copy link
Author

自定义title bar那个还有点问题,不知道为啥最大化窗口时,toggle那个方法不好使了,我是mac,你什么系统,toggle的时候能恢复最大化之前的窗口么

我是 Windows。其他系统我倒是没测试过。等我哪天看看改改提上来

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

2 participants