Skip to content

Fruit Jam Screensaver Bundle#39

Merged
relic-se merged 6 commits intorelic-se:mainfrom
cogliano:ss_bundle
Feb 5, 2026
Merged

Fruit Jam Screensaver Bundle#39
relic-se merged 6 commits intorelic-se:mainfrom
cogliano:ss_bundle

Conversation

@cogliano
Copy link
Contributor

@cogliano cogliano commented Feb 4, 2026

This pull request adds 2 new screensavers for the Fruit Jam OS, A maze generator and a star field display.

@relic-se
Copy link
Owner

relic-se commented Feb 4, 2026

Thank you for the contribution, @cogliano !

Unfortunately, I think there is a slight issue that we'll need to work through before merging this PR. It should be possible to manually configure launcher.config.js to use the specified screensaver in the bundle, but the "Screensavers" app within Fruit Jam OS doesn't currently have support for bundles outside of the built-in screensavers. The problematic code can be found here:

https://github.com/adafruit/Fruit-Jam-OS/blob/e96a25a6c740ebc976848724b8633024839a8e66/builtin_apps/Screensavers/code.py#L25-L38

This will just need a bit of enhancement to check within directories to identify screensaver modules. I think I have an idea of how best to do this, so I'll take a crack at it soon.

@cogliano
Copy link
Contributor Author

cogliano commented Feb 5, 2026

Thanks for the info. I thought it would be good to make a bundle of screensavers and have them all in one repository, since right now they are only a single file each. Also, when I add a new screensaver to the bundle in the future, I won't need to do a PR for the Fruit Jam Library. If this becomes problematic, let me know and I can split them out.

@relic-se
Copy link
Owner

relic-se commented Feb 5, 2026

No, I think this is a great approach, and I want to make sure bundling like this is supported. And yes you are right, you won't need to make any additional PRs for new releases/additions to an existing repository. Eventually, I plan to have proper support for versioning (#18), but at the moment, it'll just download the most recent.

@cogliano
Copy link
Contributor Author

cogliano commented Feb 5, 2026

Thanks. BTW, I think this library is a great idea. I think we both saw the limitations/barriers of adding community projects to the main Fruit Jam OS project. I was thinking a library type add-on would be nice and then lo and behold you created one! I'm looking forward to the production release. Let me know if you need any testers.

@relic-se
Copy link
Owner

relic-se commented Feb 5, 2026

I've authored two PRs (see above) which enable your screensaver bundle to work. I've run through the entire procedure myself (download + selection via the screensavers app) for both screensavers, and they work great! I've got the star field one running for myself, tbh. 🤣 I'm going to go ahead and merge this PR, but it will require adafruit/Fruit-Jam-OS#91 to be merged before working 100% without manual configuration.

@relic-se relic-se merged commit a50d605 into relic-se:main Feb 5, 2026
1 check passed
@relic-se
Copy link
Owner

relic-se commented Feb 5, 2026

Thanks. BTW, I think this library is a great idea. I think we both saw the limitations/barriers of adding community projects to the main Fruit Jam OS project. I was thinking a library type add-on would be nice and then lo and behold you created one!

Great minds think alike! The plan is for the development of this project will eventually lead to inclusion within Fruit Jam OS so that it will be packaged within the default installation.

I'm looking forward to the production release. Let me know if you need any testers.

There are a number of issues which I've identified but haven't yet put the legwork into (https://github.com/relic-se/Fruit_Jam_Library/issues). But the biggest issue facing the project right now and limiting its inclusion within Fruit Jam OS is the UI of browsing the database. I have a few ideas of how to improve the system, but any additional recommendations would be greatly appreciated!

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