Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

On new tab page, sometimes a topSite doesn't even have a letter #5332

Closed
cezaraugusto opened this issue Nov 1, 2016 · 6 comments
Closed

Comments

@cezaraugusto
Copy link
Contributor

Did you search for similar issues before submitting this one?
yes

Describe the issue you encountered:
On new tab page, topSites displays site's first title letter as a fallback if it couldn't find any favicon. Sometimes a letter isn't even shown. It happened for this URL.

Expected behavior:
If there's no site's favicon, letters should always appear as fallback.

  • Platform (Win7, 8, 10? macOS? Linux distro?): any

  • Brave Version: n/a

  • Screenshot if needed:
    0eae66d6-78c8-11e6-872a-21df2a90c845

@bbondy
Copy link
Member

bbondy commented Nov 2, 2016

Is there really no letter or maybe the favicon is the same color as the tile and is only 1 color?

@cezaraugusto
Copy link
Contributor Author

Not easy to reproduce but it happened a few times. Yes, no letter and there's no favicon as well. Different from accessing like https://brianbondy.com where both themeColor and favicon share the same color.

Actually this bug was pointed by you on our old newtab's PR (#3199) and I confirmed a while after. Setting here just as a reference, can be closed if there's no new reports.

@bsclifton
Copy link
Member

@bbondy @bradleyrichter would it be acceptable (if the title is blank) to use the first letter of the actual domain?

@bbondy
Copy link
Member

bbondy commented Nov 4, 2016

up to @bradleyrichter maybe that would lead to a lot of W for www., if Brad wants this you could skip that prefix though.

@bradleyrichter
Copy link
Contributor

sounds like a reasonable fallback

@bsclifton
Copy link
Member

to be consistent with the history page, I was thinking we could just use the domain (not the subdomain)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.