Add job progress and status to dock icon for macOS and Linux#1806
Merged
Add job progress and status to dock icon for macOS and Linux#1806
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Adds platform-specific job progress integration so Shotcut can surface export/job state in the OS shell (macOS Dock tile and Linux Unity launcher entry), wired from JobQueue.
Changes:
- Add macOS Dock tile progress bar + badge indicators (pause/stop/fail) APIs.
- Add Linux (Unity launcher D-Bus) progress/urgent APIs and build integration.
- Hook macOS/Linux progress updates into
JobQueueprogress + finish notifications.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| src/macos.mm | Implements Dock tile progress indicator + badge updates. |
| src/macos.h | Declares the new macOS Dock progress functions. |
| src/linuxtools.h | Declares Linux launcher progress functions. |
| src/linuxtools.cpp | Implements Unity launcher entry D-Bus progress updates. |
| src/jobqueue.cpp | Calls macOS/Linux progress APIs on job progress/finish. |
| src/CMakeLists.txt | Builds linuxtools.* on UNIX (non-Apple) targets. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Contributor
… export (#1807) * Initial plan * Add app icon as base layer in macOS dock tile contentView Agent-Logs-Url: https://github.com/mltframework/shotcut/sessions/4dfc67a1-016f-4d14-96dc-693fe562b76f Co-authored-by: ddennedy <1146683+ddennedy@users.noreply.github.com> * fix deprectation * Prevent melt subprocess from appearing in macOS Dock during export Agent-Logs-Url: https://github.com/mltframework/shotcut/sessions/ab12618a-4423-4fef-863b-caa79ab49df8 Co-authored-by: ddennedy <1146683+ddennedy@users.noreply.github.com> * Skip offscreen Qt platform for melt when GPU processing is enabled on macOS Agent-Logs-Url: https://github.com/mltframework/shotcut/sessions/83ffd684-e01b-4b15-93f2-43da8f2cb077 Co-authored-by: ddennedy <1146683+ddennedy@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: ddennedy <1146683+ddennedy@users.noreply.github.com> Co-authored-by: Dan Dennedy <dan@dennedy.org>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.