Skip to content

Move QgisMobileapp into src/app behind an AppController interface#7767

Merged
nirvn merged 1 commit into
masterfrom
app-layer-scaffold
Jul 23, 2026
Merged

Move QgisMobileapp into src/app behind an AppController interface#7767
nirvn merged 1 commit into
masterfrom
app-layer-scaffold

Conversation

@MidnightCoder-m

Copy link
Copy Markdown
Collaborator

🚀 Description

Moves QgisMobileapp out of src/core into a new src/app library (qfield_app), removing the last core → app dependency.

The only obstacle was AppInterface (which stays in core) reaching intoQgisMobileapp via qobject_cast on the QML engine. That's inverted with asmall abstract interface AppController (in core): QgisMobileapp implements it and is now handed to AppInterface explicitly, so core no longer knows the concrete app class.

What's next

  • gui layer — add src/gui (qfield_gui), move editing/attribute-form UX classes out of core
  • QML modules — adopt QML_ELEMENT
  • plugin — bundle core+gui as the QField plugin; app becomes a pure consumer

@MidnightCoder-m MidnightCoder-m self-assigned this Jul 21, 2026
@qfield-fairy

qfield-fairy commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

🍎 MacOS DMG universal builds

Download a MacOS DMG universal build of this PR for testing.
(Built from commit 24c27ba)

📱 Android builds

Download an Android arm64 build of this PR for testing.
(Built from commit 24c27ba)

Other Android architectures

🐧 Linux AppImage builds

Download a Linux AppImage build of this PR for testing.
(Built from commit 24c27ba)

🪟 Windows builds

Download a Windows build of this PR for testing and for arm64.

(Built from commit 24c27ba)

@nirvn nirvn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sweet!

@nirvn
nirvn merged commit 30d4d6d into master Jul 23, 2026
32 of 33 checks passed
@nirvn
nirvn deleted the app-layer-scaffold branch July 23, 2026 02:01
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.

3 participants