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

Update UA for Ladybird Browser (previously SerenityOS Browser) #546

Merged
merged 1 commit into from
Feb 21, 2024

Conversation

MacDue
Copy link
Contributor

@MacDue MacDue commented May 9, 2023

The browser has always been called 'Ladybird' outside SerenityOS, and now is even called Ladybird in Serenity (see SerenityOS/serenity@15211cd).

The format of the user agent has also changed since it was first added here. Given the nature of the Ladybird project it's highly unlikely there's any users of the old version, so that UA probably does not need to be preserved.

cc @linusg

@MacDue MacDue force-pushed the ladybird_browser branch 2 times, most recently from f9602f5 to bd5d1da Compare May 9, 2023 07:10
Copy link
Contributor

@linusg linusg left a comment

Choose a reason for hiding this comment

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

now is even called Ladybird in Serenity

thanks, I hate it

@MacDue
Copy link
Contributor Author

MacDue commented May 9, 2023

thanks, I hate it

Let the history books show I was on the side of 'Browser': #18735

@MacDue MacDue force-pushed the ladybird_browser branch from bd5d1da to 9f8cb81 Compare May 25, 2023 21:01
@MacDue
Copy link
Contributor Author

MacDue commented May 26, 2023

@commenthol rebased 🙂

@lbarthon lbarthon merged commit 8144b03 into ua-parser:master Feb 21, 2024
@linusg
Copy link
Contributor

linusg commented Feb 21, 2024

FWIW the regex used here has been outdated for at least 6 months by now and will no longer match: SerenityOS/serenity@4ead33d

@lbarthon
Copy link
Contributor

Checked before, noticed that it was still Ladybird/1.0 but missed that one, will file a fix

@lbarthon
Copy link
Contributor

Fix is in #573 - can you check when you have a minute @linusg?

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.

4 participants