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

Web Inspector: Upstream Safari 16.4 user agents #12898

Conversation

patrickangle
Copy link
Contributor

@patrickangle patrickangle commented Apr 19, 2023

@patrickangle patrickangle self-assigned this Apr 19, 2023
@patrickangle patrickangle added the Web Inspector Bugs related to the WebKit Web Inspector. label Apr 19, 2023
@@ -102,6 +102,7 @@ localizedStrings["0 Console warnings"] = "0 Console warnings";
localizedStrings["1 match"] = "1 match";
localizedStrings["1080p"] = "1080p";
localizedStrings["2D"] = "2D";
localizedStrings["2D (Offscreen)"] = "2D (Offscreen)";

Choose a reason for hiding this comment

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

Confused as to why this change was made?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The strings file is generated using a script, so this indicates that these strings were missing from the file previously.

/* Label for a preference that is turned off. */
localizedStrings["Off @ User Preferences Overrides"] = "Off";
localizedStrings["Offscreen2D"] = "Offscreen2D";

Choose a reason for hiding this comment

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

Ditto for these two

@patrickangle patrickangle added the merge-queue Applied to send a pull request to merge-queue label Apr 24, 2023
https://bugs.webkit.org/show_bug.cgi?id=255636
rdar://105254961

Reviewed by Devin Rousso.

* Source/WebInspectorUI/Localizations/en.lproj/localizedStrings.js:
* Source/WebInspectorUI/UserInterface/Views/OverrideDeviceSettingsPopover.js:
(WI.OverrideDeviceSettingsPopover.prototype._createUserAgentSection):

Canonical link: https://commits.webkit.org/263319@main
@webkit-commit-queue webkit-commit-queue force-pushed the eng/b255636-upstream-safari-16-4-uas branch from 0d19995 to 91a1d20 Compare April 24, 2023 17:56
@webkit-commit-queue
Copy link
Collaborator

Committed 263319@main (91a1d20): https://commits.webkit.org/263319@main

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

@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Apr 24, 2023
@webkit-commit-queue webkit-commit-queue merged commit 91a1d20 into WebKit:main Apr 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Web Inspector Bugs related to the WebKit Web Inspector.
Projects
None yet
5 participants