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 empty img tag occupy 20px size in safari. #4029

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

suzp1984
Copy link
Contributor

@suzp1984 suzp1984 commented Apr 17, 2024

the html img tag occupy 20px size in safari.

<img src='//ossrs.net/gif/v1/sls.gif?site=ossrs.net&path=/player/rtcplayer'/>

Screenshot 2024-04-17 at 9 17 07 AM

(ignore the img css warning: auto\9; it's another problem, I will file another PR.)

but, the empty img tag just occupy 1px size in chrome. So I guess it's a html compatible problem.

Screenshot 2024-04-17 at 9 46 33 AM

TRANS_BY_GPT4

@winlinvip winlinvip added the TransByAI Translated by AI/GPT. label Apr 17, 2024
@suzp1984
Copy link
Contributor Author

suzp1984 commented Apr 17, 2024

I guess the //ossrs.net/gif/v1/sls.gif?site=ossrs.net&path=/console/enindex is used for statistics & analysis purpose. but pls pay attention to the path, for some one may not so match with its real path.

e.g.

<img src='//ossrs.net/gif/v1/sls.gif?site=ossrs.net&path=/console/cnindex'/>

<img src='//ossrs.net/gif/v1/sls.gif?site=ossrs.net&path=/player/srsplayer'/>

<img src='//ossrs.net/gif/v1/sls.gif?site=ossrs.net&path=/player/rtcpublisher'/>

<img src='//ossrs.net/gif/v1/sls.gif?site=ossrs.net&path=/player/rtcpublisher'/>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TransByAI Translated by AI/GPT.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants