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

Homebox SVG logo missing in Safari due to size not specified #360

Open
6 tasks done
lmaced0 opened this issue Nov 30, 2024 · 3 comments · Fixed by #400
Open
6 tasks done

Homebox SVG logo missing in Safari due to size not specified #360

lmaced0 opened this issue Nov 30, 2024 · 3 comments · Fixed by #400
Assignees
Labels
🕷️ bug Something isn't working

Comments

@lmaced0
Copy link

lmaced0 commented Nov 30, 2024

First Check

  • This is not a feature request
  • I added a very descriptive title to this issue.
  • I used the GitHub search to find a similar issue and didn't find it.
  • I searched the documentation, with the integrated search.
  • I already read the docs and didn't find an answer.
  • I can replicate the issue inside the Demo install.

Homebox Version

0.16

What is the issue you are experiencing?

Missing width="250" height="250" in the svg portion of the code here: frontend/components/App/Logo.vue

Image

Fixed when added size:

Image

How can the maintainer reproduce the issue?

Open Homebox on Safari (iOS or macOS).

Deployment

Docker (Linux)

OS Architechture

x86_64 (AMD, Intel)

Deployment Details

No response

@lmaced0 lmaced0 added the 🕷️ bug Something isn't working label Nov 30, 2024
@lmaced0 lmaced0 changed the title SVG logo missing in Safari due to size not specified Homebox SVG logo missing in Safari due to size not specified Nov 30, 2024
@tankerkiller125
Copy link
Contributor

Of course it's Safari, it's always Safari when it comes to things like this.

@katosdev
Copy link
Contributor

Apologies for the delays @lmaced0 ! This should be fixed under #400 if you'd like to give that a go? 👍🏻

@katosdev
Copy link
Contributor

Reopening as issue still apparent. Further investigation required.

@katosdev katosdev reopened this Dec 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🕷️ bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants