TeselaGen / tg-oss

Teselagen Open Source modules
https://teselagen.github.io/tg-oss/
MIT License
39 stars 18 forks source link

UI exception in Safari browser #37

Closed hi-asan closed 12 months ago

hi-asan commented 12 months ago

@tnrich

image image

If you need any additional information, I can provide it

hi-asan commented 12 months ago
image

set letter-spacing:3px Looks normal。But in other browsers, it's not right again

tnrich commented 12 months ago

@hi-asan I'm not able to reproduce the row view issue you are seeing on safari, for me it looks fine:

image

I do see the issue where the sequence name is not being drawn in the middle of the circular view. That is due to an issue with foreignObjects and absolute positioning in safari: https://stackoverflow.com/questions/8185845/svg-foreignobject-behaves-as-though-absolutely-positioned-in-webkit-browsers

I went ahead and tried to fix that using a flexbox approach that seems to work on safari (for me). Please try that and let me know if it fixed it for you.

As for the row view issue, I'm not able to reproduce that currently. Please provide additional details and screenshots. We might need to pair to fix that one.

Thanks!

hi-asan commented 12 months ago

@tnrich Thank you for your reply

We have identified the issue and have recently updated the system to the latest Sonama 14.0. The Safari browser inside has also been updated to 17.0. Your browser version is definitely lower than this version. I have tried it on several Mac computers, and all updates to the latest version have the same issue. That is, the issue of sequence character width

hi-asan commented 12 months ago

@tnrich

Thank you for your reply

We have identified the issue and have recently updated the system to the latest Sonama 14.0. The Safari browser inside has also been updated to 17.0. Your browser version is definitely lower than this version. I have tried it on several Mac computers, and all updates to the latest version have the same issue. That is, the issue of sequence character widt

tnrich commented 12 months ago

Thanks for looking further into this @hi-asan. I'll upgrade to sonoma and then push a fix for this when I have one :)

tnrich commented 12 months ago

@hi-asan thanks for following up. I've pushed a fix for this to the latest version of ove. Let me know if that fixes it for you

hi-asan commented 11 months ago

@hi-asan thanks for following up. I've pushed a fix for this to the latest version of ove. Let me know if that fixes it for you@hi-阿桑感谢您的跟进。我已经将此问题的修复程序推送到最新版本的 ove。让我知道这是否为您解决了

The problem has been solved, thank you very much