Skip to content

Fix script popup #286

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
manuq merged 7 commits into from
Oct 23, 2024
Merged

Fix script popup #286

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
5ba3c30
Script Window: Use popup_dialog
manuq Oct 23, 2024
8288b9b
Script Window: Remove background color
manuq Oct 23, 2024
fc523e5
Script Window: Improve layout
manuq Oct 23, 2024
83b6cd2
Script Window: Change control type to CodeEdit
manuq Oct 23, 2024
b7568db
fixup! Script Window: Change control type to CodeEdit
wjt Oct 23, 2024
527bc4b
Script Window: Attempt to match editor's syntax highlighting
wjt Oct 23, 2024
778c8d7
Script Window: Don't clamp font color alpha channel to 0.65
wjt Oct 23, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 13 additions & 0 deletions addons/block_code/ui/script_window/script_window.gd
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,20 @@ var script_content: String = ""
@onready var script_label: CodeEdit = %Code


## Attempts to match the syntax highlighting for some open script in the
## editor, which is more likely to be appropriate for the editor theme
## than our hardcoded default
func _apply_editor_syntax_highlighter():
var script_editor: ScriptEditor = EditorInterface.get_script_editor()
for x in script_editor.get_open_script_editors():
var control: Control = x.get_base_editor()
if control is TextEdit:
script_label.syntax_highlighter = control.syntax_highlighter
Copy link
Contributor Author

@manuq manuq Oct 23, 2024

Choose a reason for hiding this comment

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

Awesome!

break


func _ready():
_apply_editor_syntax_highlighter()
script_label.text = script_content.replace("\t", " ")


Expand Down