-
-
Notifications
You must be signed in to change notification settings - Fork 61
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.
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
fix #345 Disable wrap mouse option if not supported #374
Conversation
I don't think we need universal logic for this, i.e. this shouldn't be an exported property. Let's have this logic in settings_menu.gd |
@@ -3,7 +3,7 @@ | |||
[ext_resource type="Script" path="res://src/ui_parts/settings_menu.gd" id="1_1gf4m"] | |||
[ext_resource type="Texture2D" uid="uid://c528knojuxbw6" path="res://visual/icons/Languages.svg" id="2_ndyp7"] | |||
[ext_resource type="Script" path="res://src/ui_elements/BetterTabContainer.gd" id="3_vslgx"] | |||
[ext_resource type="PackedScene" path="res://src/ui_elements/setting_check_box.tscn" id="4_2qeh2"] | |||
[ext_resource type="PackedScene" uid="uid://chgcy63ldo7x5" path="res://src/ui_elements/setting_check_box.tscn" id="4_2qeh2"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
idk if i should remove this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If it happened without you prompting it, don't remove it
Thanks! |
This pr does not include separate error message for that yet, im too lazy.
I haven't tested this so i just hope the display server is correct.
hopefully fixes #345
also added back the await in svg because i think its still needed? don't remember anymore but i think that was a issue again