bubkoo / html-to-image

✂️ Generates an image from a DOM node using HTML5 canvas and SVG.
MIT License
5.79k stars 544 forks source link

mat-select value is not rendered in FF #247

Closed ddzingielewski closed 2 years ago

ddzingielewski commented 2 years ago

Hi, here is what i found: I was trying to render mat-select control with selected value. It seems that htmlToImage.toPng is not able to render selected value (select is empty after rendering). I spotted it in FF 96.0.3 (it works in current Chrome and Edge)

Steps To Reproduce

Here is code that reproduces bug: https://stackblitz.com/edit/angular-ivy-gdjd5e?file=src/app/app.component.ts Choose something in "Basic mat-select" and click "Export".

It seems to be connected with inner mat-select css class:

.mat-select-value { display: table-cell; max-width: 0; width: 100%; overflow: hidden; text-overflow: ellipsis; white-space: nowrap; } After removing either: "max-width" or "overflow:hidden" the value is rendered properly. But unfortunatelly I cannot remove such styling because it leads to...well bad styling :)

It works using native select (second select in the above link)

Your Environment

biiibooo[bot] commented 2 years ago

👋 @ddzingielewski

Thanks for opening your first issue here! If you're reporting a 🐞 bug, please make sure you include steps to reproduce it. To help make it easier for us to investigate your issue, please follow the contributing guidelines.

We get a lot of issues on this repo, so please be patient and we will get back to you as soon as we can.

biiibooo[bot] commented 2 years ago

Hiya! This issue has gone quiet. Spooky quiet. 👻 We get a lot of issues, so we currently close issues after 60 days of inactivity. It’s been at least 20 days since the last update here. If we missed this issue or if you want to keep it open, please reply here. You can also add the label "not-stale" to keep this issue open! As a friendly reminder: the best way to see this issue, or any other, fixed is to open a Pull Request.

Thanks for being a part of the Antv community! 💪💯

ddzingielewski commented 2 years ago

hi, I do not see such option: "(...) You can also add the label "not-stale" to keep this issue open (...)"

How can i mark issue as "not-stale"?

czw., 3 mar 2022 o 02:17 bot-me[bot] @.***> napisał(a):

Hiya! This issue has gone quiet. Spooky quiet. 👻 We get a lot of issues, so we currently close issues after 60 days of inactivity. It’s been at least 20 days since the last update here. If we missed this issue or if you want to keep it open, please reply here. You can also add the label "not-stale" to keep this issue open! As a friendly reminder: the best way to see this issue, or any other, fixed is to open a Pull Request.

Thanks for being a part of the Antv community! 💪💯

— Reply to this email directly, view it on GitHub https://github.com/bubkoo/html-to-image/issues/247#issuecomment-1057564019, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOLC336DNHV4AEHQZVFKNITU6AHMBANCNFSM5N2MCRWQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you were mentioned.Message ID: @.***>

-- pozdrawiam Darek

biiibooo[bot] commented 2 years ago

Hiya! This issue has gone quiet. Spooky quiet. 👻 We get a lot of issues, so we currently close issues after 60 days of inactivity. It’s been at least 20 days since the last update here. If we missed this issue or if you want to keep it open, please reply here. You can also add the label "not-stale" to keep this issue open! As a friendly reminder: the best way to see this issue, or any other, fixed is to open a Pull Request.

Thanks for being a part of the Antv community! 💪💯

biiibooo[bot] commented 2 years ago

Hey again! It’s been 60 days since anything happened on this issue, so our friendly neighborhood robot (that’s me!) is going to close it. Please keep in mind that I’m only a robot, so if I’ve closed this issue in error, I’m HUMAN_EMOTION_SORRY. Please feel free to comment on this issue or create a new one if you need anything else. As a friendly reminder: the best way to see this issue, or any other, fixed is to open a Pull Request. Thanks again for being part of the Antv community! 💪💯

biiibooo[bot] commented 1 year ago

This thread has been automatically locked because it has not had recent activity. Please open a new issue for related bugs and link to relevant comments in this thread.