Skip to content

feat: Apply patches from multiple patch bundles, add GUI patch source selector #145

Open
prateek-who wants to merge 7 commits into
MorpheApp:devfrom
prateek-who:differnet-patche-source-unification
Open

feat: Apply patches from multiple patch bundles, add GUI patch source selector #145
prateek-who wants to merge 7 commits into
MorpheApp:devfrom
prateek-who:differnet-patche-source-unification

Conversation

@prateek-who
Copy link
Copy Markdown
Contributor

  • Multiple patch sources are now visible together.
  • Patch sources moves out of settings and into it's own separate place
  • Other minor UI improvements in settings dialog

@prateek-who
Copy link
Copy Markdown
Contributor Author

Liso, see how this feels. Although I like the source pill at the top of the home screen because it is minimal and clean, it looks a bit empty I feel. But what do you think? Please drop any suggestions that you feel could make this look better?

@LisoUseInAIKyrios
Copy link
Copy Markdown
Contributor

I think the pill at the top looks good. It seems the patch source is not selectable when expert mode is off.

I think the "drop apk here" could be a smaller area so more of the app list is visible.

This is a significant change (for the better) because multiple patch sources now works. Does it also work with CLI console? If not then we could add support there too. The CLI patch arguments must be positional relative to the --patches declaration, because different patch bundles can have the same patch names/options so --enable and other parameters always apply to the last --patches argument up to that point.

@LisoUseInAIKyrios LisoUseInAIKyrios changed the title feat: multi patch source unification feat: Apply patches from multiple patch bundles May 10, 2026
@LisoUseInAIKyrios LisoUseInAIKyrios changed the title feat: Apply patches from multiple patch bundles feat: Apply patches from multiple patch bundles, add GUI patch source selector May 10, 2026
@prateek-who
Copy link
Copy Markdown
Contributor Author

  1. Do we allow the non-expert mode to select different patch sources? I thought we could restrict that mode to only allow the default patch source, but if you feel it's better we allow multiple, then I'll make it the same as expert mode.
  2. Sure thing, I'll reduce the size of that apk drop zone. One other thing we could do is have the drop apk zone on the left side of the screen and show the supported apps on the right side (dropping apk anywhere on the screen works, it's just to show that zone). What do you think of this idea?
  3. Nope, CLI doesn't have it yet. Will add it now and follow the positional approach.

@LisoUseInAIKyrios
Copy link
Copy Markdown
Contributor

I think a drop zone on the right would be ok, since picking the app is usually the first step then dropping the apk is second step.

Non expert mode could allow patching from a single source (any source), while expert mode allows combining patches from multiple patch sources. This is same behavior as Manager.

@prateek-who
Copy link
Copy Markdown
Contributor Author

  • Overhaul the expert mode home screen
  • Added new theme (Deepspace)
  • Non-expert mode now allows to select a single patch source, following the manager's style.

See how this runs and feels. CLI still needs work. Will start that once you feel the GUI feels right.

…e-unification

# Conflicts:
#	src/main/kotlin/app/morphe/gui/ui/components/SettingsDialog.kt
@LisoUseInAIKyrios
Copy link
Copy Markdown
Contributor

The GUI looks great.

Can the drop zone be moved to the right of the app selection? I think it may feel more natural since a wizard style flow usually navigated left to right.

Otherwise I think is ready.

@prateek-who
Copy link
Copy Markdown
Contributor Author

Now see how this feels. Please tell me if something feels off.

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.

2 participants