-
Notifications
You must be signed in to change notification settings - Fork 250
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
Associate tasks to specific workspaces and follow changes automatically #259
base: master
Are you sure you want to change the base?
Conversation
|
||
try: | ||
self.bus = dbus.SessionBus() | ||
except: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please only catch the specific exceptions you expect.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As I'm not familiar with dbus, I copied this from idle.py, but all other places in hamster where a session bus is retrieved are unprotected, so I'm taking out the try/except here.
Thanks for your contribution! To merge this I would definitely like:
Let me know if you need any pointers. |
Thanks for your feedback! I agree on the settings part but I first wanted to know if it had a chance of getting in. I also want to add a "auto resume" setting to the lock screen part. For the save/restore part, I wanted to start by adding a simple API for it, but indeed the implementation needs to be solid, maybe some additions to the storage object could help: is there a way to reuse the existing fact object to start a new slot? The gnome-shell extension has buttons to restart one of the previous tasks, this code could be shared. |
Thank you for your feedback, it is much appreciated. Project hamster and its various sub-components is currently undergoing some major changes. We prepare the introduction of a rewritten codebase for most of the underlying functionality. A direct consequence of this is that it is unlikely that any open/new bugs within the current/old codebase will be fixed (unless someone steps up and offers to do so) as most resources currently available will be invested in making the rewrite prime time ready. If you are still interested in working on the new codebase (repositories: hamster-lib/cli/gtk/dbus) we would be most thrilled. Please feel free to either open a new issue with the relevant repository and/or join the discussion on the mailinglist. Thanks for your interest and support! Eric. |
Yes, in view of #494, this isn't going to be implemented any time soon. We'd need to get the desktop integration back first. But that doesn't mean it's not a good idea. |
Aside from leaving it open, another option would be to open a new feature request issue in whatever project/fork undertakes reviving desktop integration and linking back here. AFAIK closed requests don't disappear, they are just less visible. |
You might as well say "invisible". Who scans closed issues? We should differentiate between "will never implement" and "won't implement just now". |
W00ps, i was a little overzealous in cleanup @ederag's comment. Turns out it still relevant, since there is a "Pending check" for CLA that can probably never realized. Here's the deleted comment:
Sorry for the noise. |
I added a simple mechanism to save/restore previous tasks and used it to:
It still lacks proper setting (and their GUI) but it "works for me" :)