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 'popup_centered_minsize()' not working #29760

Closed

Conversation

YeldhamDev
Copy link
Member

No description provided.

@akien-mga akien-mga added this to the 3.2 milestone Jun 14, 2019
@akien-mga
Copy link
Member

Is it popup_centered_minsize() that needs this fix, or popup_centered() as a whole as done in #24907?

@YeldhamDev
Copy link
Member Author

@akien-mga I just tested that PR and it doesn't fix the issue on this one.

@akien-mga
Copy link
Member

Would be nice if you described the issue this PR is fixing then :)

@YeldhamDev
Copy link
Member Author

YeldhamDev commented Jun 14, 2019

After analysing the function better, I found that it's actually not intended to that what I thought it did (actually, its job seems quite useless, but alright), so my "fix" is moot. Closing it.

@YeldhamDev YeldhamDev closed this Jun 14, 2019
@YeldhamDev YeldhamDev deleted the fix_popup_centered_minsize branch June 14, 2019 12:22
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