Tomasz Chabora
Tomasz Chabora
Duplicating big complex functions makes them more difficult to keep in sync in case a modification is needed.
Sooo I tried to test if it works and used this hack (editor node) ```C++ case EDIT_REDO: { ResourceSaver::set_uid("res://icon.svg", ResourceUID::get_singleton()->text_to_id("test1")); ResourceSaver::set_uid("res://new_theme.tres", ResourceUID::get_singleton()->text_to_id("test2")); ResourceSaver::set_uid("res://new_theme.res", ResourceUID::get_singleton()->text_to_id("test3")); ``` The method has no effect...
Why won't `text_to_id()` work though? The UID is stored as ``. But the number UIDs work. Except for `new_theme.tres`, which for whatever reason gets saved as `new_theme.tres.uidren` (with correct UID)...
And what about the uidren thing? It was created by `set_uid()` so it's probably related. (I tested with newly created, empty Theme)
It's still there, here's MRP: [Uidren.zip](https://github.com/godotengine/godot/files/10373197/Uidren.zip) Note that I'm testing on newest master, so you might need to rebase to get the same result.
Maybe it's system-specific? I'm on Windows.  I even checked out to your branch to make sure and it's still the same.
It doesn't 🙃 And the UID is wrong now 🙃🙃 `[gd_resource type="Theme" load_steps=0 format=3 uid="uid://no"]` EDIT: Also I did this test ```C++ if (!all_ok) { print_line(">", fw->get_error()); return ERR_CANT_CREATE; }...
Tested with this code ```GDScript func _process(delta: float) -> void: if Input.is_action_just_pressed("ui_accept"): if DisplayServer.window_get_mode() != DisplayServer.WINDOW_MODE_FULLSCREEN: DisplayServer.window_set_mode(DisplayServer.WINDOW_MODE_FULLSCREEN) else: DisplayServer.window_set_mode(DisplayServer.WINDOW_MODE_WINDOWED) #DisplayServer.window_set_mode(DisplayServer.WINDOW_MODE_WINDOWED) ``` If you maximize the window and switch to fullscreen,...
Caused by #70909 It's the same issue as the one fixed by the reverted PR, i.e. pressing Animation will reload inspector and reopen all properties, triggering unwanted editors.
Can't reproduce in beta 17. Please attach a *real* MRP.