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

fix: firefox layout breaking #45

Merged
merged 3 commits into from Jul 13, 2020
Merged

fix: firefox layout breaking #45

merged 3 commits into from Jul 13, 2020

Conversation

anuraghazra
Copy link
Owner

@anuraghazra anuraghazra commented Jul 13, 2020

fixes #45

@anuraghazra anuraghazra linked an issue Jul 13, 2020 that may be closed by this pull request
@anuraghazra
Copy link
Owner Author

anuraghazra commented Jul 13, 2020

@saurabhdaware can you check the updates?

here's live development preview (without icons)

with icons

without fix:

@cjdenio
Copy link

cjdenio commented Jul 13, 2020

It's definitely better, but still a bit off.

image

@anuraghazra
Copy link
Owner Author

@cjdenio Yes i checked it, i have no idea why tho, its working fine locally. ff has lot of issues with SVGs 😩 weird

@cjdenio
Copy link

cjdenio commented Jul 13, 2020

Weird, it works perfectly when icons are turned on.
image

@anuraghazra
Copy link
Owner Author

Huh! very weird

@anuraghazra
Copy link
Owner Author

Oh got it, for some reason user-agent isn't recognized

@saurabhdaware
Copy link

Yes I have same output as @cjdenio

@anuraghazra
Copy link
Owner Author

Ahh got it, the problem is that in the request headers i'm getting github-camo instead of firefox's ua which makes sense because the API is hit from server not directly from browser, any ideas to fix this? @saurabhdaware @cjdenio

@saurabhdaware
Copy link

saurabhdaware commented Jul 13, 2020

Can we fix that on frontend? https://stackoverflow.com/questions/5378559/including-javascript-in-svg

Edit:
So after getting response it will send javascript along with it that will fix that space on frontend

P.S. Super noob with SVGs so I have no idea if this makes sense or not

@anuraghazra
Copy link
Owner Author

Not that easy i assume, the SVGs wont gonna be rendered as SVG or pure HTML elements by github, github will automatically upload the svg to their camo.githubusercontent cdn and then render that as image.

@saurabhdaware
Copy link

ahh yes right

@anuraghazra
Copy link
Owner Author

@saurabhdaware can you check again it should be fixed :D

@cjdenio
Copy link

cjdenio commented Jul 13, 2020

image

Looks perfect!

@anuraghazra
Copy link
Owner Author

Awesome! thanks @cjdenio

@saurabhdaware
Copy link

uh oh I turned off my laptop and sleeping now :( so probably by tomorrow morning

@anuraghazra
Copy link
Owner Author

No issues @saurabhdaware @cjdenio sent the screenshot its working fine now, thanks for helping out, and reporting the bug. ❤️

@anuraghazra anuraghazra merged commit 014f710 into master Jul 13, 2020
@anuraghazra anuraghazra added bug Something isn't working. design Issues, fixes related to design or alignments. labels Jul 19, 2020
@anuraghazra anuraghazra mentioned this pull request Jul 24, 2020
@anuraghazra anuraghazra deleted the fix-firefox branch October 31, 2020 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working. design Issues, fixes related to design or alignments.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Layout is little off in Ubuntu Firefox
3 participants