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

Add standard vertical spacing to some drawers. #284

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

Conversation

nitz
Copy link

@nitz nitz commented Nov 16, 2021

A few drawers needed a little breathing room from the controls that followed them, in particular InfoBox and the help box drawn from DrawDefaultPropertyAndHelpBox. Added EditorGUIUtility.standardVerticalSpacing to the heights for those instances. As well, used EditorGUIUtility.standardVerticalSpacing for the height of controls inside a reorderable list, instead of the hardcoded value 4.

The difference in drawing is minimal, sure, but it lets those infoboxes line up in a way that doesn't make them feel like they're crowding the drawer underneath them.

image

image

A few drawers needed a little breathing room from the controls that followed them, in particular InfoBox and the help box drawn from `DrawDefaultPropertyAndHelpBox`. Added EditorGUIUtility.standardVerticalSpacing to the heights for those instances. As well, used EditorGUIUtility.standardVerticalSpacing for the height of controls inside a reorderable list, instead of the hardcoded value 4.
@nitz nitz force-pushed the add-standard-vertical-spacing branch from f72b2d7 to e229506 Compare June 14, 2022 15:48
@nitz
Copy link
Author

nitz commented Jun 14, 2022

Updated this PR to not conflict with base.

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.

1 participant