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

Linux support #19

Closed
1 of 4 tasks
tomlin7 opened this issue May 6, 2022 · 2 comments · Fixed by #23
Closed
1 of 4 tasks

Linux support #19

tomlin7 opened this issue May 6, 2022 · 2 comments · Fixed by #23
Assignees
Labels
enhancement New feature or request

Comments

@tomlin7
Copy link
Owner

tomlin7 commented May 6, 2022

The editor is currently not supporting linux operating system.

Bugs

Bugs identified so far.

  • keysym <Control-`> doesn't work
  • cmd is not recognized in linux
  • tk-extrafont library have trouble installing in linux

Fixes:

  • System data class #21
    A class to hold all needed data of the working machine. This shall include operating system, version, python version, etc.
  • Configure new keysym to replace <Control-`> when on linux.
  • instead of cmd use bash in linux (make choice based on platform.system())
  • add fixes to avoid the error while installing tk-extrafont in linux
@tomlin7 tomlin7 added the enhancement New feature or request label May 6, 2022
@cid0rz
Copy link
Collaborator

cid0rz commented May 6, 2022

nice! I will continue investigating. Didnt want to open the issue myself, so thanks :)

@tomlin7
Copy link
Owner Author

tomlin7 commented May 6, 2022

#22 Added SysInfo class, so for system info now it is possible to do base.sysinfo.os, base.sysinfo.version, base.sysinfo.python_version, etc.

@tomlin7 tomlin7 mentioned this issue May 7, 2022
tomlin7 pushed a commit that referenced this issue Oct 28, 2023
fix: dont import all of tkinter.constants
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants