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

Fix file menu separator #2593

Merged
merged 1 commit into from
Dec 12, 2018
Merged

Conversation

HebaruSan
Copy link
Member

Problem

There's supposed to be a separator line between Audit recommendations and Exit, but it doesn't appear:

image

Cause

Wanting to be frugal, I re-used toolStripSeparator3 rather than creating a new separator, since the actual separator object stores essentially no data.

That apparently doesn't work.

Changes

Now we use a new toolStripSeparator7 object instead. This makes the separator show up:

image

@HebaruSan HebaruSan added Bug Something is not working as intended Easy This is easy to fix GUI Issues affecting the interactive GUI Pull request labels Dec 3, 2018
@politas politas merged commit 13219a0 into KSP-CKAN:master Dec 12, 2018
politas added a commit that referenced this pull request Dec 12, 2018
@HebaruSan HebaruSan deleted the fix/file-menu-separator branch December 12, 2018 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is not working as intended Easy This is easy to fix GUI Issues affecting the interactive GUI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants