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

Moved scrollbar on leader list so it won't hide stats. #78

Closed
wants to merge 3 commits into from

Conversation

JohanO
Copy link

@JohanO JohanO commented Jul 14, 2016

Fix Issue #77

@lprichar
Copy link
Member

Hey JohanO, thanks so much for all the recent contributions! Everything looks pretty good on this fix, however it looks like I'm getting some new screen clipping issues on the people detail page:

image

Are you seeing that in your environment?

@JohanO
Copy link
Author

JohanO commented Jul 16, 2016

Oh, I didn't notice that one, but yes, I am seeing that as well.
I increased the size of the list of leaders to make room for the scrollbar. Thats probably the cause of the clipping. I will look into it.

@JohanO
Copy link
Author

JohanO commented Jul 17, 2016

The Designer in Visual Studio are playing trix on me. I reverted my previous change and hand coded the .designer.cs files. Now it looks much better.
2016-07-17

@lprichar
Copy link
Member

Sorry, forgot to merge this in. Thanks so much @JohanO!

@lprichar lprichar closed this Jul 27, 2016
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.

2 participants