Skip to content

Implement custom font folder management in preferences - #43

Open
PAL123 wants to merge 2 commits into
samytichadou:masterfrom
PAL123:PAL123-Implement-custom-font-folder-support
Open

PAL123 wants to merge 2 commits into
samytichadou:masterfrom
PAL123:PAL123-Implement-custom-font-folder-support

Conversation

@PAL123

@PAL123 PAL123 commented Apr 24, 2026

Copy link
Copy Markdown

PAL123 added 2 commits April 24, 2026 11:31
Added functionality to manage custom font folders in the add-on preferences, including operators to add and remove folders.
Added a custom wrapper to append user-defined font folders from addon preferences.

@samytichadou samytichadou left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thanks a lot for your work, a few refinements, but this will be a nice addition to the addon :)

Comment thread addon_prefs.py
# --- UI für die Custom Folders ---
box = layout.box()
box.label(text="Custom Font Folders:", icon='FILE_FOLDER')

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

add an aligned column to keep the layout more compact

Comment thread addon_prefs.py
@@ -1,10 +1,47 @@
import bpy

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Globally remove german (comment and description) and replace it by english

Comment thread load_fonts.py
print("FONTSELECTOR --- OS not supported")



Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

to me this is not the proper way to do this, it should not be either custom or os folders. We should keep both (OS folder, custom folders), and not have error handling to replace the original function.
I think a get_folders function should call os folders AND custom folders.
This allows to have custom behavior in the future, with a bool prop for example, to use only one or the other. And this would be easier to read an understand.

Tell me if i'm not clear ! :)

And can you replace german by english ?

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