FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

migrating Preferences class into stand-alone version by AhmedMagedC · Pull Request #1209 · processing/processing4 · GitHub

/ processing4 Public

migrating Preferences class into stand-alone version - #1209

Closed
AhmedMagedC wants to merge 1 commit into
processing:mainfrom
AhmedMagedC:standalone-preference
ConversationCommits (1)ChecksFiles changed
Closed

migrating Preferences class into stand-alone version#1209
AhmedMagedC wants to merge 1 commit into
processing:mainfrom
AhmedMagedC:standalone-preference

Conversation

AhmedMagedC commented Aug 15, 2025

Copy link
Copy Markdown
Contributor

Resolves #1104

Changes

  • breaks the Preferences class into two classes: app.utils.Preferences and app.Preferences
  • app.utils.Preferences class is the stand-alone version making the loading and saving of preferences independent
  • app.Preferences depends on app.utils.Preferences for the main functionalties (loading, saving, getting a value) but also adds other app dependent functionalties (fonts, language, ...)
  • gracefully handles exceptions by adding a centralized custom Exception: PreferencesException
  • loading of default.txt is now via JAR resource system

Todo

  • the SettingFolder location depends on the Base class, which forces you to explicitly set the location when using app.utils.Preferences. We can decouple it from Base class making it independt of app module
  • Adding callback events/listeners when a preference changes
  • Tests that will declare and test the functionality of Preferences
  • A Github Action that will run the Tests on all supported platforms

diyaayay mentioned this pull request Jul 2, 2026
Open
49 tasks

catilac commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

Hi @AhmedMagedC we are finally getting to this. And it's so late. Is this something you still have the bandwidth to help with? I'm so sorry this took so long for us to get back to this. A lot of stuff has changed, and your work will need to be re-based. @diyaayay is interested in taking this over the line, but I want to confirm with you first.

AhmedMagedC commented Jul 30, 2026

Copy link
Copy Markdown
Contributor Author

Hi @AhmedMagedC we are finally getting to this. And it's so late. Is this something you still have the bandwidth to help with? I'm so sorry this took so long for us to get back to this. A lot of stuff has changed, and your work will need to be re-based. @diyaayay is interested in taking this over the line, but I want to confirm with you first.

Hi @catilac , Sorry I won't be available to help with this.

catilac commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

Thanks @AhmedMagedC for the quick reply. Will close this for now.

catilac closed this Jul 31, 2026
diyaayay added a commit to diyaayay/processing4 that referenced this pull request Aug 6, 2026
add standalone Preferences to app:utils
e924d99 
storage half of app.Preferences: defaults from classpath, load/save,
platform-specific keys. throws instead of showing dialogs; init split
so the app can insert its own defaults between stages. follows processing#1209.
diyaayay mentioned this pull request Aug 6, 2026
Open
1 task
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
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Making Preferences standalone

2 participants

Back | FazBrowse Home | New Git URL