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

Replace iOS "Add to Home Screen" icon ("apple-touch-icon") #9694

Closed
1 of 2 tasks
andrewcleveland opened this issue Jun 11, 2021 · 1 comment
Closed
1 of 2 tasks

Replace iOS "Add to Home Screen" icon ("apple-touch-icon") #9694

andrewcleveland opened this issue Jun 11, 2021 · 1 comment
Assignees

Comments

@andrewcleveland
Copy link

andrewcleveland commented Jun 11, 2021

(Edit: Changed from feature request to bug report template.)

Please confirm you have done the following before posting your bug report:

Describe the bug
iOS uses the Snipe-IT logo as the app icon when using the "Add to Home Screen" function in Safari from the Snipe-IT Dashboard page.
The image logo.png does poorly as an app icon for two reasons:

  1. iOS puts the transparent logo over black, resulting in black text on a black background.
  2. It doesn't fit the square icon aspect ratio, so only part of the logo appears.

To Reproduce
Steps to reproduce the behavior:

  1. In Safari, go to https://demo.snipeitapp.com/
  2. Tap the share button at the bottom of the screen.
  3. Tap "Add to Home Screen".

Expected behavior
The Snipe-IT logo appears as the logo for the app.

Screenshots
Current app icon:
screenshot of transparent icon result

Possible replacement apple-touch-icon image:
apple-touch-icon

App icon with replacement image:
screenshot of new icon result

Server (please complete the following information):

  • Snipe-IT Version 5.1.5, from Docker image snipe/snipe-it:v5.1.5
  • OS: Ubuntu 18.04.5 LTS
  • Web Server: Apache 2.4.29
  • PHP Version 7.2.24-0ubuntu0.18.04.7

Smartphone (please complete the following information):

  • Device: iPhone 7
  • OS: iOS 14.6
  • Browser: Safari
  • Version 14.6
  • User-agent: Mozilla/5.0 (iPhone; CPU iPhone OS 14_6 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/14.1.1 Mobile/15E148 Safari/604.1

Error Messages
None

Additional context

  • Is this a fresh install or an upgrade? Upgrade
  • What OS and web server you're running Snipe-IT on: Using Docker image
  • What method you used to install Snipe-IT Docker
  • Include what you've done so far in the installation, and if you got any error messages along the way. No error messages
  • Indicate whether or not you've manually edited any data directly in the database No

The "apple-touch-icon" link tag in resources/views/layouts/default.blade.php specifies the icon image.
First added in 6736b1c in response to #7238.
The feature is documented here: Configuring Web Applications

@welcome
Copy link

welcome bot commented Jun 11, 2021

👋 Thanks for opening your first issue here! If you're reporting a 🐞 bug, please make sure you include steps to reproduce it. 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants