Merged
Conversation
AlexanderVanhee
force-pushed
the
addons-rework-2
branch
from
3146f9f to
efdc814
Compare
Collaborator
|
Awesome, will take a look in a sec!!! |
Addon group IDs were dropped the first time addons came in because they came before applications, only getting saved the second time when the initial cache got replaced and entry groups for apps already existed
Collaborator
|
looks good now except please correct the code to follow the style rules. I want bz-application.c to stay nice static functions need to be declared at the top of the file |
Collaborator
Author
|
OK, I fixed the style issues I saw. |
Collaborator
|
Thanks! |
AsciiWolf
reviewed
Apr 9, 2026
| src/bz-content-provider.c | ||
| src/bz-context-tile.blp | ||
| src/bz-context-tile.c | ||
| src/bz-context-tile-callbacks.blp |
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
3 participants
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.
Revamps add-on systems, by improving the add-on dialog and putting them in entry groups.
The add-on dialog now has like a mini version of the full view, which is designed around the metadata that these add-ons provide. It also fixes the issue where you would have a half-empty add-ons dialog when only one add-on is available (quite common) for a specific app, as we can now just show the new full-view.
Opening the
appstream:addon-idURL will also show the dialog with the new full view, making it possible to link users to an add-on from within another application (or even from the Flathub site).Add-ons are now also put in entry groups, so it should now be possible to install from multiple remotes like with normal apps. It also makes display filters work and allows add-ons to be marked EOL. Only consumed properties are actually put in these groups to save some ram.