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

Test Ladybird #55

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Test Ladybird #55

wants to merge 2 commits into from

Conversation

shlyakpavel
Copy link

This is a WIP PR for testing Ladybird browser
It currently works, but needs some extra work to make things nice
image
This is done as part of LadybirdBrowser/ladybird#2306
This also requires LadybirdBrowser/ladybird#2323 to get nice 500x500 pngs

@shlyakpavel shlyakpavel force-pushed the ladybird branch 3 times, most recently from 9f14db2 to 58bd010 Compare November 13, 2024 20:01
Updates ladybird results with the current master, as well as some Firefox and Safari tests with current releases
@shlyakpavel
Copy link
Author

shlyakpavel commented Nov 14, 2024

output-4
27.5% , not bad

@RazrFalcon
Copy link
Collaborator

That's the easy "25%". It would only go downhill from here. But yeah, a good result for a new project. In 5 years would be close to 75% at this pace.

Not worth adding to the main test suite yet. Let's wait for at least 50-60%.

@shlyakpavel
Copy link
Author

@RazrFalcon I hope you don't mind leaving this WIP PR here for future reference. I'll update results here from time to time, and I really hope we reach 50% soon. We're beating 1592684/1860522 subtests on WPT today, how hard can fixing SVGs be?)
I expect a lot of new tests to pass when this list is done and LadybirdBrowser/ladybird#2248 is merged

@RazrFalcon
Copy link
Collaborator

Sure, we can leave it.

how hard can fixing SVGs be?

Well, I would be very curious to know that 😄

@gmta
Copy link

gmta commented Nov 15, 2024

Biggest issue is probably also taken care of soon: LadybirdBrowser/ladybird#2357

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants