Add Live Info Panel - #1002
Conversation
|
Let's change the menu name to I loaded this up and got these errors, so I couldn't test it. |
44eeccf to
5b22da7
Compare
Apologies - i didn't commit the new scene file! Now fixed and renamed the menu button. |
These are all fixed
I wish I'd thought of this!
I can't get it to look exactly right - but it is now much closer. Apparently (according to co-pilot) there is some trickery happening to adjust the contrast curve depending on the pixels behind which I don't think we can reasonably replicate.
I have changed my approach as I had a large node tree with HBoxes in Vboxes, name Labels and value Labels. It is now presented in a PanelContainer with a single Label with all of the results combined in a single string. I believe this is how the information_panel works too. I have addressed the above and it looks closer to the information panel to me.
Yes, mouse pos and region still return results and the other values display nan.
The panel expands, and I have removed parentheses and adjusted decimal precision.
Weird... I loaded in 4.7.1, and fixed it by setting the vertical flag to shrink_end, not sure why that didn't behave the same between versions. It now appears reliably in the left bottom corner in 4.5.2, 4.6.3 and 4.7.1 |
5b22da7 to
ee364a5
Compare
It seems to me, the name of the style is "Information3dViewport" And you can get the styling directly from the editor like this, which is common practice. That way if it changes because of the theme or code updates, ours will continue to match. |
ee364a5 to
abbaf17
Compare
|
That works perfectly 👌 |
|
Looks and works much better now. Solid work. When the region tool is selected the current position is off and the results are a bit off. So I've asterisked the position, and kept the only valid data: the region location. I've replaced nan with
Base and overlay are ints, so I've removed the period. I've also padded them with 0s so the pipes are fixed in place. Sadly padding with space doesn't work between 1 and 2 digit texture ids, even if done manually.
I also opened the live panel scene and saved it in 4.5.2 without making any changes. Godot gave the file some updates. Please do the following and we can merge it:
You can squash my changes. |
bba714f to
d7d502b
Compare
| func _ready() -> void: | ||
| _editor_settings = EditorInterface.get_editor_settings() | ||
| if not _editor_settings.has_setting("terrain3d/config/live_info_panel_enabled"): | ||
| _editor_settings.set_setting("terrain3d/config/live_info_panel_enabled", enabled) |
There was a problem hiding this comment.
It's better if you use plugin.set_setting() and maybe plugin.setup_editor_settings()
There was a problem hiding this comment.
Do you mean to check if the setting exists in setup_editor_settings() and if not set a default value? I prefer it being part of the new script, self contained etc... Let me know what you think and/or if I misunderstood.
d7d502b to
7e59130
Compare
7e59130 to
595f3b6
Compare
|
Excellent work, thank you. |








This adds the live info panel as discussed on Discord.
Fixes #984