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

Allow moving the editor's bottom panel to the right #93901

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vintnarni
Copy link
Contributor

@vintnarni vintnarni commented Jul 3, 2024

Closes (partially) #26229.

EditorNode-splited supported version of #36040.

This lets people make better use of their display if it's wide enough.

Like the original, the shortcut is Shift + F10.

Unlike the original, it disallows moving to the right if the screen/window's width isn't at least 1800px, as well as forbid closing the subpanel when at the right and an ability to collapse the current tab by holding Shift key while also at the right.

Preview

For the second clip, i'm using 1300 as the minimum width for demonstration.

right-bottom-panel-preview.mp4

Note

  • Window uses _propagate_window_notification on editor builds in _update_viewport_size to fix broken NOTIFICATION_WM_SIZE_CHANGED, it will be removed as soon as the first task was completed (see below).

TODOs

  • Figure how to set up NOTIFICATION_WM_SIZE_CHANGED properly without using _propagate_window_notification.
  • Save the setting across restarts.
  • Better icon(s), currently uses generic arrow icon.

This lets people make better use of their display if it's wide enough.

Co-authored-by: Calinou <[email protected]>
Co-authored-by: MewPurPur <[email protected]>
@vintnarni vintnarni force-pushed the right-bottom-panel branch from 3f75320 to 3d385b7 Compare July 4, 2024 07:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants