Skip to content

Conversation

@Zebastjan
Copy link

  • Implement D-Bus service using dbus-next for cleaner Python integration
  • Add shell script toggle-syllablaze.sh for KDE global shortcuts
  • Update install.py to copy shortcut script to user's .local/bin
  • Add dbus-next and qasync dependencies to requirements.txt

This change allows users to control Syllablaze using global keyboard shortcuts in KDE. When setting up shortcuts, users must add the script as an 'Application' rather than a 'Command' for proper functionality.

- Implement D-Bus service using dbus-next for cleaner Python integration
- Add shell script toggle-syllablaze.sh for KDE global shortcuts
- Update install.py to copy shortcut script to user's .local/bin
- Add dbus-next and qasync dependencies to requirements.txt

This change allows users to control Syllablaze using global keyboard
shortcuts in KDE. When setting up shortcuts, users must add the script
as an 'Application' rather than a 'Command' for proper functionality.
@Zebastjan
Copy link
Author

Thank you so much for creating this wonderful application. It's extraordinarily helpful. It's exactly the sort of thing that we've been wanting for a long, long time. So, I hope the changes in adding the facility for a keyboard shortcut will be helpful and appreciated. And if there's anything else we can do. There's two possibilities that we note.

One is that for whatever reason, when we click on the functionality to download additional models to select, it doesn't seem to be working. That's something we could have a look at.

The second thing we note is that main.py is on the order of 750 lines, which is a bit unwieldy. So, we might think about refactoring that. However, we don't want to scribble all over your code. So, you know, let us know if you have any ideas or something that we could take a look at that you might appreciate.

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.

1 participant