Skip to content

Conversation

uncleLukie
Copy link
Contributor

Summary:

This pull request resolves critical errors and warnings when using Dialogic with the newly released Godot 4.5. The primary fixes include updating the deprecated SVGTexture to the new DPITexture and correcting invalid resource UIDs.

Changes:

  • SVGTexture to DPITexture: In Godot 4.5, the SVGTexture class has been renamed to DPITexture. This was causing an error (Cannot get class 'SVGTexture') when loading scenes that used the old class name. This has been fixed in addons/dialogic/Editor/Events/Fields/field_file.tscn by replacing all instances of SVGTexture with DPITexture.

  • Invalid UID Fixes: The timeline_editor.tscn file contained incorrect UIDs for the timeline_editor.gd and shortcut_popup.gd scripts. This has been resolved by updating the UIDs to match the actual file UIDs, which prevents any potential issues with script loading and scene integrity.

The `SVGTexture` class has been replaced with `DPITexture` in `field_file.tscn` to resolve the "Cannot get class 'SVGTexture'" error. All corresponding resource IDs have been updated accordingly.

Additionally, incorrect UIDs for `timeline_editor.gd` and `shortcut_popup.gd` in `timeline_editor.tscn` have been corrected to match the actual file UIDs, resolving potential scene loading issues.
@CakeVR
Copy link
Collaborator

CakeVR commented Sep 18, 2025

If DPITexture is not supported in older Godot versions, Dialogic would cease to work for a great range of users. Therefore, that class name change is not a solution we can accept.

@CakeVR CakeVR added Compatibility Breaking ⚠️ This PR breaks compatibility in some way Bug 🐞 Something isn't working labels Sep 18, 2025
They are loaded at runtime anyways. This means they should continue to work correctly in every release.
@Jowan-Spooner
Copy link
Member

Hey @uncleLukie thanks for the PR! As cake said both DPI Texture and SVGTexture (which was only available for a short while during the 4.5 dev phase aren't recognized by older godot versions. But I made a commit just removing the icons from the scene as they are loaded at runtime anyways. I guess godot changed what texture type get_theme_icon() returns. This way this should be good to go.

Thanks again!

@Jowan-Spooner Jowan-Spooner changed the title Godot 4.5 Compatibility: SVGTexture to DPITexture and UID Fixes Godot 4.5 Compatibility: UID Fixes and remove SVGTexture references Sep 19, 2025
@Jowan-Spooner Jowan-Spooner merged commit 27b1fac into dialogic-godot:main Sep 19, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug 🐞 Something isn't working Compatibility Breaking ⚠️ This PR breaks compatibility in some way
Development

Successfully merging this pull request may close these issues.

3 participants