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: honor dialog.showMessageBox()'s Icon argument on Linux (backport: 3-0-x) #15342

Merged
merged 3 commits into from Oct 23, 2018

Conversation

Projects
None yet
2 participants
@trop
Contributor

trop bot commented Oct 23, 2018

Backport of #15326

See that PR for details.

Notes: Honor dialog.showMessageBox()'s icon argument on Linux

ckerr added some commits Oct 22, 2018

Don't call gtk_widget_show_all() on popup dialog.
Fixes #15317.

Notes: Fixed incorrect display of some GtkMessageDialog icons.

The issue is caused because GtkMessageDialog contains an icon widget
which is not shown when there's no associated icon. Our call to
`gtk_widget_show_all()` overrides this, showing the uninitialized
icon widget.

This PR fixes the issue by calling `gtk_widget_show()` where needed
and removing use of `gtk_widget_show_all()` in the message dialog.

@trop trop bot requested a review from electron/reviewers as a code owner Oct 23, 2018

@trop trop bot referenced this pull request Oct 23, 2018

Merged

fix: honor dialog.showMessageBox()'s Icon argument on Linux #15326

4 of 6 tasks complete

@trop trop bot added 3-0-x backport labels Oct 23, 2018

@codebytere codebytere merged commit 8dce401 into electron:3-0-x Oct 23, 2018

13 of 14 checks passed

ci/circleci: electron-mas-x64 Your tests failed on CircleCI
Details
Absolute Zero
Semantic Pull Request ready to be squashed
Details
WIP ready for review
Details
appveyor: win-ia32-testing-pr AppVeyor build succeeded
Details
appveyor: win-x64-testing-pr AppVeyor build succeeded
Details
ci/circleci: electron-linux-arm Your tests passed on CircleCI!
Details
ci/circleci: electron-linux-arm-test Your tests passed on CircleCI!
Details
ci/circleci: electron-linux-arm64 Your tests passed on CircleCI!
Details
ci/circleci: electron-linux-arm64-test Your tests passed on CircleCI!
Details
ci/circleci: electron-linux-ia32 Your tests passed on CircleCI!
Details
ci/circleci: electron-linux-x64 Your tests passed on CircleCI!
Details
ci/circleci: electron-osx-x64 Your tests passed on CircleCI!
Details
release-notes Release notes found
@release-clerk

This comment has been minimized.

release-clerk bot commented Oct 23, 2018

Release Notes Persisted

Honor dialog.showMessageBox()'s icon argument on Linux

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment