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.

Sign up for GitHub

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

Jump to bottom

✨ Feature: 为Linux系统适配桌面图标栏（Tray） #603

Merged
merged 2 commits into from
Dec 29, 2020
Merged

✨ Feature: 为Linux系统适配桌面图标栏（Tray） #603

merged 2 commits into from
Dec 29, 2020

Conversation

peiyuanix
Copy link
Contributor

No description provided.

@peiyuanix peiyuanix closed this Dec 27, 2020
@peiyuanix peiyuanix reopened this Dec 27, 2020
Molunerfinn
Molunerfinn requested changes Dec 28, 2020
View reviewed changes
src/main/apis/app/system/index.ts Outdated Show resolved Hide resolved
src/main/apis/app/system/index.ts Show resolved Hide resolved
src/main/apis/app/system/index.ts Show resolved Hide resolved
src/main/lifeCycle/index.ts Outdated Show resolved Hide resolved
src/main/apis/app/system/index.ts Show resolved Hide resolved
@peiyuanix
Copy link
Contributor Author

更新：

  1. 去除if (process.platform === 'darwin' || process.platform === 'win32' || process.platform === "linux") 等无效代码
  2. 重新添加误删的适用于Mac的Tray拖拽相关操作代码

@Molunerfinn
Copy link
Owner

下班后有时间看看，不要催哈

@peiyuanix
Copy link
Contributor Author

peiyuanix commented Dec 28, 2020 •
edited
Loading

已测试环境：

  1. Ubuntu20.04+Unity桌面环境下Tray表现正常
  2. Manjaro20.02+i3桌面环境下Tray表现正常
  3. Arch最新版（滚动更新）+i3桌面环境下Tray表现正常

@Molunerfinn Molunerfinn changed the title ✨ Feature: 为Ubuntu系统适配桌面图标栏（Tray） ✨ Feature: 为Linux系统适配桌面图标栏（Tray） Dec 29, 2020
@Molunerfinn Molunerfinn merged commit 0fe3ade into Molunerfinn:dev Dec 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Molunerfinn Molunerfinn Molunerfinn approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@peiyuanix @Molunerfinn