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

REGRESSION(262797@main): Menu permanently opened on github.com #16309

Merged
merged 1 commit into from
Aug 2, 2023

Conversation

carlosgcampos
Copy link
Contributor

@carlosgcampos carlosgcampos commented Aug 2, 2023

476e8fb

REGRESSION(262797@main): Menu permanently opened on github.com
https://bugs.webkit.org/show_bug.cgi?id=258290

Reviewed by Michael Catanzaro.

In 262797@main popover attribute was disabled for apple apps linkking to
the older SDKs, but also for non-apple ports. What happens in github now
is that the popover is never hidden because display: revert ends up in
block. This is because the popover css, which sets display: none when
popover-open is not present, is not included for any element.

* Source/WebKit/Shared/WebPreferencesDefaultValues.cpp:
(WebKit::defaultPopoverAttributeEnabled): Enable it by default for non-apple ports again.

Canonical link: https://commits.webkit.org/266510@main

6a83f3e

Misc iOS, tvOS & watchOS macOS Linux Windows
βœ… πŸ§ͺ style βœ… πŸ›  ios βœ… πŸ›  mac βœ… πŸ›  wpe βœ… πŸ›  wincairo
βœ… πŸ›  ios-sim βœ… πŸ›  mac-AS-debug βœ… πŸ§ͺ wpe-wk2
βœ… πŸ§ͺ webkitperl βœ… πŸ§ͺ ios-wk2 βœ… πŸ§ͺ api-mac βœ… πŸ›  gtk
βœ… πŸ§ͺ ios-wk2-wpt βœ… πŸ§ͺ gtk-wk2
βœ… πŸ§ͺ api-ios βœ… πŸ§ͺ mac-wk2 βœ… πŸ§ͺ api-gtk
βœ… πŸ›  tv βœ… πŸ§ͺ mac-AS-debug-wk2
βœ… πŸ›  tv-sim
βœ… πŸ›  πŸ§ͺ merge βœ… πŸ›  watch
βœ… πŸ›  watch-sim

@carlosgcampos carlosgcampos self-assigned this Aug 2, 2023
@carlosgcampos carlosgcampos added the WebKitGTK Bugs related to the Gtk API layer. label Aug 2, 2023
Copy link
Contributor

@mcatanzaro mcatanzaro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My hero!

fyi: @nt1m

@mcatanzaro mcatanzaro added the merge-queue Applied to send a pull request to merge-queue label Aug 2, 2023
https://bugs.webkit.org/show_bug.cgi?id=258290

Reviewed by Michael Catanzaro.

In 262797@main popover attribute was disabled for apple apps linkking to
the older SDKs, but also for non-apple ports. What happens in github now
is that the popover is never hidden because display: revert ends up in
block. This is because the popover css, which sets display: none when
popover-open is not present, is not included for any element.

* Source/WebKit/Shared/WebPreferencesDefaultValues.cpp:
(WebKit::defaultPopoverAttributeEnabled): Enable it by default for non-apple ports again.

Canonical link: https://commits.webkit.org/266510@main
@webkit-commit-queue
Copy link
Collaborator

Committed 266510@main (476e8fb): https://commits.webkit.org/266510@main

Reviewed commits have been landed. Closing PR #16309 and removing active labels.

@webkit-commit-queue webkit-commit-queue merged commit 476e8fb into WebKit:main Aug 2, 2023
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Aug 2, 2023
@carlosgcampos carlosgcampos deleted the gtk-github-popover branch August 2, 2023 13:15
@nt1m
Copy link
Member

nt1m commented Aug 2, 2023

Sorry about that, thanks for fixing !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WebKitGTK Bugs related to the Gtk API layer.
Projects
None yet
5 participants