Skip to content
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

Move "Virtual Spaces" preference to "Editor -> Features" #3643

Merged
merged 1 commit into from
Nov 2, 2023

Conversation

b4n
Copy link
Member

@b4n b4n commented Oct 22, 2023

"Editor -> Display" is very crowded already, and virtual spaces are not merely a display feature, but rather a tool for editing.

Close #3605.

"Editor -> Display" is very crowded already, and virtual spaces are not
merely a display feature, but rather a tool for editing.

Close geany#3605.
@b4n b4n added this to the 2.1 (or something?) milestone Oct 22, 2023
@techee
Copy link
Member

techee commented Oct 23, 2023

Looks good - I just tested this PR and it works as described (no way for a human being to review the XML diff itself).

In the future we could introduce some tab like "Behavior" which is so generic-sounding we'd be free to place there just anything :-).

@b4n
Copy link
Member Author

b4n commented Oct 23, 2023 via email

@elextr
Copy link
Member

elextr commented Oct 23, 2023

In the future we could introduce some tab like "Behavior" which is so generic-sounding we'd be free to place there just anything :-).

What, "General" isn't general enough ;-P

<property name="receives-default">False</property>
<property name="tooltip-text" translatable="yes">Do not show virtual spaces</property>
<property name="use-underline">True</property>
<property name="active">True</property>
Copy link
Member

Choose a reason for hiding this comment

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

This seems new but probably added by Glade automatically and we set an explicit default value in the code anyway, so it's fine.

Copy link
Member

@eht16 eht16 left a comment

Choose a reason for hiding this comment

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

LGTM

@b4n b4n merged commit 2dc8c22 into geany:master Nov 2, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Prefs: move "virtual spaces" to the "Features" tab
4 participants