-
Notifications
You must be signed in to change notification settings - Fork 628
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
Error displaying images with "Languages Used (By File Size)" #57
Comments
Thanks for the details in the bug report! I was able to replicate this in Chrome on Desktop (see attached video). The issue appears to be with some internal padding value in the CSS. I will leave this open for documentation until I can fix the problem. 2021-12-13.03-54-35.mp4 |
For anyone who is considering working on this, but is intimidated or unsure how to get started, I would be happy to help! Feel free to post any questions in this thread. A fix would involve you forking the repository, making some changes, committing and pushing those to your fork, and making a pull request on GitHub that we can work together on until it's ready to merge. At the end, your GitHub profile will be listed as a "contributor" on this repo! I expect (i.e., guess) that the places where the file might need to be changed are probably some or all of:
If you want to try fixing this, you should go through the values in the list above and play around with them in the Thanks in advance to anyone who takes the time to help out with this! |
Hey @jstrieb I was wondering if I could attempt to help with this one? |
Hey @Froderick-Fronkensteen, please do! If you can get it fixed, make a PR, and we can work to try it on some devices and get it merged in. Thanks for your interest in the project! |
Hey @jstrieb, if this issue hasn't been resolved yet, I would like to work on it. |
Hi @vidit21srivastava! This issue has not yet been resolved, and I would appreciate your help with it. |
Hey @jstrieb , first I was thinking that the "height" of "foreignObject" is causing the problem. But later I got a strange thing. What is this class doing here? By removing it's "height" everything is ok. Please checkout that. |
Error description photo
Device used in the photo
Browser version used in the photo
The text was updated successfully, but these errors were encountered: