Mac player #343
Mac player #343
Conversation
|
hi,
so far i have not tested:
|
📝 WalkthroughWalkthroughA new "mac-player" plugin feature for tmux was introduced, including comprehensive documentation, a Bash script for macOS media control and display, and integration into the Dracula theme script. The documentation and scripts detail configuration, player detection, remote control keybindings, and visual customization for both native and browser-based music players. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant tmux
participant dracula.sh
participant mac-player.sh
participant AppleScript
participant MediaApp
User->>tmux: Uses tmux with Dracula theme
tmux->>dracula.sh: Requests plugin status
dracula.sh->>mac-player.sh: Executes for "mac-player"
mac-player.sh->>AppleScript: Queries active media player
AppleScript->>MediaApp: Retrieves playback info
MediaApp-->>AppleScript: Returns track/status
AppleScript-->>mac-player.sh: Returns info
mac-player.sh->>dracula.sh: Provides formatted track/status
dracula.sh->>tmux: Displays music info in status bar
User->>tmux: (Optional) Uses keybinding for media control
tmux->>mac-player.sh: Triggers remote control command
mac-player.sh->>AppleScript: Sends play/pause/next/back
AppleScript->>MediaApp: Executes control command
✨ Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 4
🧹 Nitpick comments (5)
scripts/dracula.sh (2)
59-59: Suppress ShellCheck warning for unused purple variableThe
purplecolor variable is used indirectly via indirection (${!colors[0]}), so add a ShellCheck directive to silence SC2034.- purple="#b166cc" +# shellcheck disable=SC2034 + purple="#b166cc"
273-275: Align and harden the mac-player plugin blockEnsure consistent indentation, quote variables in tests, and prevent word splitting in the command substitution.
- elif [ $plugin = "mac-player" ]; then - IFS=' ' read -r -a colors <<< $(get_tmux_option "@dracula-mac-player-colors" "purple dark_gray") - script="#($current_dir/mac-player.sh)" + elif [ "$plugin" = "mac-player" ]; then + IFS=' ' read -r -a colors <<< "$(get_tmux_option "@dracula-mac-player-colors" "purple dark_gray")" + script="#(${current_dir}/mac-player.sh)"scripts/mac-player.sh (3)
96-96: Use standard AppleScript comparison operatorAppleScript may not recognize the Unicode '≥'. Replace it with the ASCII operator
>=for compatibility.- if (count of titleParts) ≥ 2 then + if (count of titleParts) >= 2 then
192-193: Improve cache file path scopingUsing a fixed
/tmp/tmux_mac_player_cachecan lead to conflicts between sessions and users. Generate a unique cache file per session or user.-local cache_file="/tmp/tmux_mac_player_cache" +cache_file="${TMPDIR:-/tmp}/tmux_mac_player_${TMUX_PANE:-$$}"
224-227: Use file modification time for cache staleness
stat -f %creturns inode change time. Use%mfor modification time to reliably detect staleness.- if [ ! -f "$cache_file" ] || [ $(($(date +%s) - $(stat -f%c "$cache_file"))) -ge "$RATE" ]; then + if [ ! -f "$cache_file" ] || [ $(( $(date +%s) - $(stat -f%m "$cache_file") )) -ge "$RATE" ]; then
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
docs/CONFIG.md(15 hunks)scripts/dracula.sh(2 hunks)scripts/mac-player.sh(1 hunks)
🧰 Additional context used
🪛 Shellcheck (0.10.0)
scripts/dracula.sh
[warning] 59-59: purple appears unused. Verify use (or export if used externally).
(SC2034)
[warning] 274-274: Quote this to prevent word splitting.
(SC2046)
🪛 LanguageTool
docs/CONFIG.md
[uncategorized] ~219-~219: Possible missing comma found.
Context: .... _make sure to include the compact-alt widget as you won't be able to switch out of n...
(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~316-~316: It seems likely that a singular genitive (’s) apostrophe is missing.
Context: ...orking directory, where the path of the users home directory will be shortened to ~...
(AI_HYDRA_LEO_APOSTROPHE_S_XS)
[uncategorized] ~339-~339: It appears that hyphens are missing in the adjective “up-to-date”.
Context: ...` Set symbol to use for when branch is up to date with HEAD ```bash # default is ✓. Avoi...
(UP_TO_DATE_HYPHEN)
[grammar] ~382-~382: The operating system from Apple is written “macOS”.
Context: ... - partial support for apple m-chips on MacOS. - partial support for amd and intel on...
(MAC_OS)
[uncategorized] ~436-~436: It appears that hyphens are missing in the adjective “up-to-date”.
Context: ...` Set symbol to use for when branch is up to date with HEAD ```bash #default is ✓.Avoid ...
(UP_TO_DATE_HYPHEN)
[grammar] ~514-~514: The modal verb ‘should’ requires the verb’s base form.
Context: ...c - Browser Players ( Active Tab should consists of this players ): - youtube.com/watc...
(MD_BASEFORM)
[grammar] ~514-~514: The singular determiner ‘this’ may not agree with the plural noun ‘players’. Did you mean “these”?
Context: ...Players ( Active Tab should consists of this players ): - youtube.com/watch - op...
(THIS_NNS)
[typographical] ~575-~575: Usually, there’s no comma before “if”.
Context: ...nly display the wifi you're connected to, if it provides internet access!** You can...
(IF_NO_COMMA)
[style] ~577-~577: Consider a more concise word here.
Context: ...** You can use different hosts to ping in order to check for a wifi or wired connection. I...
(IN_ORDER_TO_PREMIUM)
[grammar] ~608-~608: The verb form ‘are’ does not seem to match the subject ‘name’.
Context: ...rface. The most common interfaces name are eth0 for a wired connection and `wlan...
(SUBJECT_VERB_AGREEMENT)
[grammar] ~695-~695: Probable usage error. Use “and” after ‘both’.
Context: ... combination, both of the local machine as well as when connected via ssh. To output noth...
(BOTH_AS_WELL_AS)
🪛 markdownlint-cli2 (0.17.2)
docs/CONFIG.md
327-327: Emphasis used instead of a heading
null
(MD036, no-emphasis-as-heading)
594-594: Fenced code blocks should have a language specified
null
(MD040, fenced-code-language)
671-671: Fenced code blocks should have a language specified
null
(MD040, fenced-code-language)
711-711: Fenced code blocks should have a language specified
null
(MD040, fenced-code-language)
733-733: Emphasis used instead of a heading
null
(MD036, no-emphasis-as-heading)
735-735: Fenced code blocks should have a language specified
null
(MD040, fenced-code-language)
777-777: Fenced code blocks should have a language specified
null
(MD040, fenced-code-language)
783-783: Fenced code blocks should have a language specified
null
(MD040, fenced-code-language)
793-793: Fenced code blocks should have a language specified
null
(MD040, fenced-code-language)
scripts/mac-player.sh
Outdated
| if [[ $len > $std ]]; then | ||
| result="${str:0:std}" | ||
| result+="..." | ||
| else | ||
| result=$str | ||
| fi |
There was a problem hiding this comment.
Fix numeric comparison and substring expansion in sliceTrack
Use a numeric comparison (-gt or arithmetic context) and proper variable expansion for the substring length.
function sliceTrack() {
local str="$1"
local std="$2"
- local len=${#str}
- if [[ $len > $std ]]; then
- result="${str:0:std}"
+ local len=${#str}
+ if (( len > std )); then
+ result="${str:0:$std}"
result+="..."
else
result=$str📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| if [[ $len > $std ]]; then | |
| result="${str:0:std}" | |
| result+="..." | |
| else | |
| result=$str | |
| fi | |
| if (( len > std )); then | |
| result="${str:0:$std}" | |
| result+="..." | |
| else | |
| result=$str | |
| fi |
🤖 Prompt for AI Agents
In scripts/mac-player.sh around lines 152 to 157, the comparison between $len
and $std uses a string comparison operator (>) instead of a numeric comparison,
and the substring expansion uses an unquoted variable which can cause issues.
Replace the string comparison with a numeric comparison using -gt or arithmetic
context, and ensure the substring length variable is properly expanded with
braces and no quotes to correctly slice the string.
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (2)
docs/CONFIG.md (2)
25-25: Duplicate: TOC anchor fix applied
The[mac-player](#mac-player---up)link now matches the heading anchor. No further changes needed.
548-551: Fix typo in tmux option name
The option is declared as@dracule-mac-player-appbut should use thedraculaprefix.- set -g @dracule-mac-player-app "Music" + set -g @dracula-mac-player-app "Music"
🧹 Nitpick comments (2)
docs/CONFIG.md (2)
511-517: Clarify browser support and correct grammar
The phrase “Active Tab should consists of this players” is ungrammatical and doesn’t call out browser requirements.Proposed diff:
### mac-player - [up](#table-of-contents) This widget and script displays music information provided by the native macOS players. The current supported apps are: - Native Players: - Spotify - Music - Apple Music - Browser Players ( Active Tab should consists of this players ): - - youtube.com/watch - - open.spotify.com + - Browser Players (active tab must match one of these URLs and requires Google Chrome or Safari): + - youtube.com/watch + - open.spotify.com
540-543: Specify code block language
The remote‐activation block lacks a language hint. For consistency, change the opening fence to```bash.- ``` + ```bash set -g @dracula-mac-player-remote true</blockquote></details> </blockquote></details> <details> <summary>📜 Review details</summary> **Configuration used: CodeRabbit UI** **Review profile: CHILL** **Plan: Pro** <details> <summary>📥 Commits</summary> Reviewing files that changed from the base of the PR and between 405a4d08edb00bd247926c00c0a7e8b75a0181c8 and 3e86f3a5faa42a240673c2ad6429ef4f425706a6. </details> <details> <summary>📒 Files selected for processing (2)</summary> * `docs/CONFIG.md` (15 hunks) * `scripts/mac-player.sh` (1 hunks) </details> <details> <summary>🚧 Files skipped from review as they are similar to previous changes (1)</summary> * scripts/mac-player.sh </details> <details> <summary>🧰 Additional context used</summary> <details> <summary>🪛 LanguageTool</summary> <details> <summary>docs/CONFIG.md</summary> [uncategorized] ~219-~219: Possible missing comma found. Context: .... _make sure to include the compact-alt widget as you won't be able to switch out of n... (AI_HYDRA_LEO_MISSING_COMMA) --- [uncategorized] ~316-~316: It seems likely that a singular genitive (’s) apostrophe is missing. Context: ...orking directory, where the path of the users home directory will be shortened to `~`... (AI_HYDRA_LEO_APOSTROPHE_S_XS) --- [uncategorized] ~339-~339: It appears that hyphens are missing in the adjective “up-to-date”. Context: ...` Set symbol to use for when branch is up to date with HEAD ```bash # default is ✓. Avoi... (UP_TO_DATE_HYPHEN) --- [grammar] ~382-~382: The operating system from Apple is written “macOS”. Context: ... - partial support for apple m-chips on MacOS. - partial support for amd and intel on... (MAC_OS) --- [uncategorized] ~436-~436: It appears that hyphens are missing in the adjective “up-to-date”. Context: ...` Set symbol to use for when branch is up to date with HEAD ```bash #default is ✓.Avoid ... (UP_TO_DATE_HYPHEN) --- [grammar] ~514-~514: The modal verb ‘should’ requires the verb’s base form. Context: ...c - Browser Players ( Active Tab should consists of this players ): - youtube.com/watc... (MD_BASEFORM) --- [grammar] ~514-~514: The singular determiner ‘this’ may not agree with the plural noun ‘players’. Did you mean “these”? Context: ...Players ( Active Tab should consists of this players ): - youtube.com/watch - op... (THIS_NNS) --- [typographical] ~581-~581: Usually, there’s no comma before “if”. Context: ...nly display the wifi you're connected to, if it provides internet access!** You can... (IF_NO_COMMA) --- [style] ~583-~583: Consider a more concise word here. Context: ...** You can use different hosts to ping in order to check for a wifi or wired connection. I... (IN_ORDER_TO_PREMIUM) --- [grammar] ~614-~614: The verb form ‘are’ does not seem to match the subject ‘name’. Context: ...rface. The most common interfaces name are `eth0` for a wired connection and `wlan... (SUBJECT_VERB_AGREEMENT) --- [grammar] ~701-~701: Probable usage error. Use “and” after ‘both’. Context: ... combination, both of the local machine as well as when connected via ssh. To output noth... (BOTH_AS_WELL_AS) </details> </details> <details> <summary>🪛 markdownlint-cli2 (0.17.2)</summary> <details> <summary>docs/CONFIG.md</summary> 327-327: Emphasis used instead of a heading null (MD036, no-emphasis-as-heading) --- 600-600: Fenced code blocks should have a language specified null (MD040, fenced-code-language) --- 677-677: Fenced code blocks should have a language specified null (MD040, fenced-code-language) --- 717-717: Fenced code blocks should have a language specified null (MD040, fenced-code-language) --- 739-739: Emphasis used instead of a heading null (MD036, no-emphasis-as-heading) --- 741-741: Fenced code blocks should have a language specified null (MD040, fenced-code-language) --- 783-783: Fenced code blocks should have a language specified null (MD040, fenced-code-language) --- 789-789: Fenced code blocks should have a language specified null (MD040, fenced-code-language) --- 799-799: Fenced code blocks should have a language specified null (MD040, fenced-code-language) </details> </details> </details> <details> <summary>🔇 Additional comments (4)</summary><blockquote> <details> <summary>docs/CONFIG.md (4)</summary> `4-4`: **Skip trivial formatting change** The inserted blank line after the “## Table of Contents” heading is a minor style tweak. --- `505-523`: **Approve mac-player section introduction** The new plugin overview is clear, follows the existing style, and correctly lists supported native and browser players along with a note on refresh rate. --- `528-532`: **Approve icon customization example** The play/pause icon variables are documented correctly and follow the same pattern as other plugins. --- `536-539`: **Approve length customization example** The `@dracula-mac-player-length` option is properly documented with a default value. </details> </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit for review status -->
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
docs/CONFIG.md (1)
505-565: Reviewmac-playerplugin documentation
- Verify that each
@dracula-mac-player-*option name and its default value in this section aligns exactly with the defaults and flags implemented inscripts/mac-player.sh.- Grammar nit: “This widget and script displays music information…” → “This widget and script display music information…”
- UX suggestion: consider renaming
@dracula-mac-player-remote-ppto something more descriptive (e.g.,@dracula-mac-player-play-pause) to match the style of other keybinding options.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
docs/CONFIG.md(15 hunks)
🧰 Additional context used
🪛 LanguageTool
docs/CONFIG.md
[uncategorized] ~219-~219: Possible missing comma found.
Context: .... _make sure to include the compact-alt widget as you won't be able to switch out of n...
(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~316-~316: It seems likely that a singular genitive (’s) apostrophe is missing.
Context: ...orking directory, where the path of the users home directory will be shortened to ~...
(AI_HYDRA_LEO_APOSTROPHE_S_XS)
[uncategorized] ~339-~339: It appears that hyphens are missing in the adjective “up-to-date”.
Context: ...` Set symbol to use for when branch is up to date with HEAD ```bash # default is ✓. Avoi...
(UP_TO_DATE_HYPHEN)
[grammar] ~382-~382: The operating system from Apple is written “macOS”.
Context: ... - partial support for apple m-chips on MacOS. - partial support for amd and intel on...
(MAC_OS)
[uncategorized] ~436-~436: It appears that hyphens are missing in the adjective “up-to-date”.
Context: ...` Set symbol to use for when branch is up to date with HEAD ```bash #default is ✓.Avoid ...
(UP_TO_DATE_HYPHEN)
[typographical] ~581-~581: Usually, there’s no comma before “if”.
Context: ...nly display the wifi you're connected to, if it provides internet access!** You can...
(IF_NO_COMMA)
[style] ~583-~583: Consider a more concise word here.
Context: ...** You can use different hosts to ping in order to check for a wifi or wired connection. I...
(IN_ORDER_TO_PREMIUM)
[grammar] ~614-~614: The verb form ‘are’ does not seem to match the subject ‘name’.
Context: ...rface. The most common interfaces name are eth0 for a wired connection and `wlan...
(SUBJECT_VERB_AGREEMENT)
[grammar] ~701-~701: Probable usage error. Use “and” after ‘both’.
Context: ... combination, both of the local machine as well as when connected via ssh. To output noth...
(BOTH_AS_WELL_AS)
🪛 markdownlint-cli2 (0.17.2)
docs/CONFIG.md
327-327: Emphasis used instead of a heading
null
(MD036, no-emphasis-as-heading)
600-600: Fenced code blocks should have a language specified
null
(MD040, fenced-code-language)
677-677: Fenced code blocks should have a language specified
null
(MD040, fenced-code-language)
717-717: Fenced code blocks should have a language specified
null
(MD040, fenced-code-language)
739-739: Emphasis used instead of a heading
null
(MD036, no-emphasis-as-heading)
741-741: Fenced code blocks should have a language specified
null
(MD040, fenced-code-language)
783-783: Fenced code blocks should have a language specified
null
(MD040, fenced-code-language)
789-789: Fenced code blocks should have a language specified
null
(MD040, fenced-code-language)
799-799: Fenced code blocks should have a language specified
null
(MD040, fenced-code-language)
🔇 Additional comments (2)
docs/CONFIG.md (2)
4-4: Minor whitespace adjustment for formatting; no action required.
25-25: TOC entry for new plugin
The mac-player entry correctly follows the existing TOC pattern and matches its section anchor (#mac-player---up).
|
Hey @Theoreticallyhugo, Sorry for the miscommunication on the tasks, I did some digging and found out that firefox is also not supported by applescript so I placed the empty tasks for anyone who has something to add on that. So to conclude on the first point, yes it can be merged.
I've documented the variable I forgot about it.
This is may be due to my formatting but I've fixed where necessary.
This is caused by activating the remote flag due to the default option being spotify. I got this a lot when I turned it into Music as my app but after sometime it stopped. Maybe if a lot people can test it and then we can find more issues that arise.
Oh yeah. I was so into the working part and forgot to proof read but I've removed the else. For the other browsers, they don't have applescript support and the only option is javascript injections and some weird hacky UI scripting which is bad. Tidal are not focused on applescript support so we can disregard those options at this point. |
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
docs/CONFIG.md (2)
526-538: Document customization options concisely
Icon (@dracula-mac-player-play-icon,@dracula-mac-player-pause-icon) and length (@dracula-mac-player-length) options are clear. You may want to note default values inline (e.g.,♪for play,❚❚for pause, length 25) for quick reference.
540-550: Clarify activation and app selection
@dracula-mac-player-remoteand@dracula-mac-player-appare well-documented. Consider validating that the app value is one of"Music"or"Spotify"in the script and mention fallback behavior in the docs.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
docs/CONFIG.md(15 hunks)scripts/mac-player.sh(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- scripts/mac-player.sh
🧰 Additional context used
🪛 LanguageTool
docs/CONFIG.md
[uncategorized] ~219-~219: Possible missing comma found.
Context: .... _make sure to include the compact-alt widget as you won't be able to switch out of n...
(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~316-~316: It seems likely that a singular genitive (’s) apostrophe is missing.
Context: ...orking directory, where the path of the users home directory will be shortened to ~...
(AI_HYDRA_LEO_APOSTROPHE_S_XS)
[uncategorized] ~339-~339: It appears that hyphens are missing in the adjective “up-to-date”.
Context: ...` Set symbol to use for when branch is up to date with HEAD ```bash # default is ✓. Avoi...
(UP_TO_DATE_HYPHEN)
[grammar] ~382-~382: The operating system from Apple is written “macOS”.
Context: ... - partial support for apple m-chips on MacOS. - partial support for amd and intel on...
(MAC_OS)
[uncategorized] ~436-~436: It appears that hyphens are missing in the adjective “up-to-date”.
Context: ...` Set symbol to use for when branch is up to date with HEAD ```bash #default is ✓.Avoid ...
(UP_TO_DATE_HYPHEN)
[typographical] ~581-~581: Usually, there’s no comma before “if”.
Context: ...nly display the wifi you're connected to, if it provides internet access!** You can...
(IF_NO_COMMA)
[style] ~583-~583: Consider a more concise word here.
Context: ...** You can use different hosts to ping in order to check for a wifi or wired connection. I...
(IN_ORDER_TO_PREMIUM)
[grammar] ~614-~614: The verb form ‘are’ does not seem to match the subject ‘name’.
Context: ...rface. The most common interfaces name are eth0 for a wired connection and `wlan...
(SUBJECT_VERB_AGREEMENT)
[grammar] ~701-~701: Probable usage error. Use “and” after ‘both’.
Context: ... combination, both of the local machine as well as when connected via ssh. To output noth...
(BOTH_AS_WELL_AS)
🪛 markdownlint-cli2 (0.17.2)
docs/CONFIG.md
327-327: Emphasis used instead of a heading
null
(MD036, no-emphasis-as-heading)
600-600: Fenced code blocks should have a language specified
null
(MD040, fenced-code-language)
677-677: Fenced code blocks should have a language specified
null
(MD040, fenced-code-language)
717-717: Fenced code blocks should have a language specified
null
(MD040, fenced-code-language)
739-739: Emphasis used instead of a heading
null
(MD036, no-emphasis-as-heading)
741-741: Fenced code blocks should have a language specified
null
(MD040, fenced-code-language)
783-783: Fenced code blocks should have a language specified
null
(MD040, fenced-code-language)
789-789: Fenced code blocks should have a language specified
null
(MD040, fenced-code-language)
799-799: Fenced code blocks should have a language specified
null
(MD040, fenced-code-language)
🔇 Additional comments (10)
docs/CONFIG.md (10)
4-4: skip formatting-only change
Blank line insertion for spacing.
25-25: Confirm correct TOC anchor for mac-player
The entry now links to#mac-player---up, matching the section heading's anchor.
92-92: skip formatting-only change
Blank line insertion after Powerline heading.
100-100: skip formatting-only change
Blank line insertion beforeEnable edge iconscode block.
151-151: skip formatting-only change
Blank line insertion before left-icon padding example.
219-219: skip formatting-only change
Formatting adjustment before compact-alt instructions.
505-514: Addmac-playerplugin section header
The new section clearly introduces the macOS music player widget. Consider specifying any prerequisites (e.g., tmux version or AppleScript permission).
511-517: List supported native and browser players accurately
Good clarity on native (Spotify, Music) and browser (YouTube, Spotify web) apps. If future support is planned for additional URLs or browsers, note it here.
518-523: Detail the remote-player feature scope
The supported remote players mirror the display section—Spotify and Apple Music. Consider specifying that the remote control also uses AppleScript and may require enabling “Accessibility” for terminal apps.
552-564: Include default and custom keybinding guidance
Default (P,R,N) and customizable remote keybindings are described. For completeness, you could mention how to unbind or reassign if conflicts occur.
Docstrings generation was requested by @ethancedwards8. * #343 (comment) The following files were modified: * `scripts/dracula.sh` * `scripts/mac-player.sh`
Sorry for the delay, this is based on the enhancment pull request #336
The following things were done based on the tasks assigned to myself:
Contains a remote feature but the apps only supported are: