-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
Report-in-Firefox fixes #1489
Report-in-Firefox fixes #1489
Conversation
will have a look ^^ if @ebidel doesn't beat me to it 😃 |
Also note that this patch doesn't fix the checkbox issue; I thought I'd leave this for later. |
Let's do checkboxes in another PR. |
@@ -1,62 +1,3 @@ | |||
<style> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we restore these files and keep the related styles within the partials? That doesn't seem necessary for this PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done. I just feel that having random styles in many places isn't good. But, yeah, it was unrelated change to this PR.
Firefox doesn't display the SVGs unless they are URL encoded.
Opening this since I'm not sure when I'll have more time to fix the remaining issues.
If merged, fixes #1124.
/CC @ebidel @wardpeet: feel free to push to my branch. I also kept the original patch which URL encoded the SVGs and worked everywhere but it's a little messy solution.
Also, there's a discrepancy between Chrome and Firefox: in Firefox the badges don't look properly vertically aligned but in Chrome they look fine. Not sure how to fix that only for Firefox.