fix(mango_layouts): use correct mmsg syntax for layout retrieval and dispatch - #451
Closed
DNI9 wants to merge 1 commit into
Closed
fix(mango_layouts): use correct mmsg syntax for layout retrieval and dispatch#451DNI9 wants to merge 1 commit into
DNI9 wants to merge 1 commit into
Conversation
…dispatch mmsg does not output JSON or support 'get all-monitors' and 'dispatch' subcommands. Switched to 'mmsg -g' with awk for parsing active monitor layout, and 'mmsg -d setlayout' for applying layouts.
Contributor
|
This pull request was automatically closed because its description no longer contains Missing:
Please add the items listed above back to the description, keeping their exact wording, then |
Contributor
|
CC @ezequielgk |
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.
Plugin
ezequiel/mango_layoutsplugin.toml)What it does
Fixes broken
mmsgIPC commands. Replaces invalidmmsg get all-monitors | jqwithmmsg -g | awkto parse active monitor layout, and fixesmmsg dispatchtommsg -d setlayout.External dependencies
mmsg, awk, jq (existing dependencies).
Testing
Tested reading the layout and updating it via
mmsgcommands.Screenshots / Videos
N/A - bug fix for background command
Checklist
idafter the/inplugin.tomlexactly.plugin.toml,README.md,thumbnail.webp, andtranslations/en.json.README.mdfollows the README template, documents every entry id and dependency, and includes exact panel IPC commands and launcher prefixes where applicable.thumbnail.webpwith the thumbnail generator.versionfollows semver and is bumped in this PR;plugin_apiis the oldest API level this plugin requires.catalog.toml; CI generates it.Code review attestation
licensedeclared inplugin.toml.