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

Undo changes from #242 #263

Merged
merged 1 commit into from
Sep 28, 2022
Merged

Undo changes from #242 #263

merged 1 commit into from
Sep 28, 2022

Conversation

itamarst
Copy link
Contributor

@itamarst itamarst commented Sep 27, 2022

Fixes #261

I'm not reproducing #241 anymore, may just be luck though (it was intermittent). If this makes it start happening again I'll try again with some other approach, but I feel like having wrong width is a lot worse than occasional extra padding on the bottom.

@codecov
Copy link

codecov bot commented Sep 27, 2022

Codecov Report

Base: 90.21% // Head: 90.21% // No change to project coverage 👍

Coverage data is based on head (14ae2a3) compared to base (900f255).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #263   +/-   ##
=======================================
  Coverage   90.21%   90.21%           
=======================================
  Files          19       19           
  Lines        4179     4179           
=======================================
  Hits         3770     3770           
  Misses        409      409           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@jonhoo jonhoo merged commit 358693a into jonhoo:main Sep 28, 2022
@jonhoo
Copy link
Owner

jonhoo commented Sep 28, 2022

Thanks! Released in 0.11.8 🎉

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.

More Firefox rendering issues
3 participants