-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
White background in favicon #22981
Comments
cc @owncloud/designers |
I guess this is because of the move to the ICO for IE compatibility. |
ref #18039 |
I think we could rework the whole favicon stuff for 9.1 because IE up to 10 isn't supported anymore in 9.1 #15567 (comment) |
So, we could drop .ico again? |
Yes. |
Recreate the .ico with transparent background should be enough. At least for 9.0.x and 8.2.x. The one in my screenshot is created from the png. I used a random ico generator on the internet. Didn't know if there is a better method to create it. |
@MorrisJobke as far as I know you introduced the .ico support – can you remove it again? |
With OwnCloud 9.1it looks much better. If you look for it, you can see it, but it is barely noticeable. |
Hey, this issue has been closed because the label (This is an automated comment from GitMate.io.) |
looks fine to me with 10.0.5 RC3. |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Steps to reproduce
Expected behaviour
Actual behaviour
Server configuration
Operating system: Debian Jessie
Web server: nginx 1.6.2
Database: MySQL
PHP version: 5.6
ownCloud version: (see ownCloud admin page) 9.0
Updated from an older ownCloud or fresh install: Updated
Where did you install ownCloud from: package manager (apt-get)
Signing status (ownCloud 9.0 and above):
The text was updated successfully, but these errors were encountered: