Have ability to mix named folders with only favicons (no names shown for icons)

brave / browser-laptop

[DEPRECATED] Please see https://github.com/brave/brave-browser for the current version of Brave

https://www.brave.com

Other

7.94k stars 975 forks source link

Have ability to mix named folders with only favicons (no names shown for icons) #9975

Open charlescyl opened 7 years ago

charlescyl commented 7 years ago

Did you search for similar issues before submitting this one? Yes
Describe the issue you encountered: I would like to request the favicon to be updated, seems like when i import bookmark the updated bookmarks do not update the favicon.

Bookmark option Instead of having to choose can we just choose how we want it, can't seem to have a mix of both words bookmark bar or favicon only.

Platform (Win7, 8, 10? macOS? Linux distro?): Win 7
Brave Version (revision SHA): Latest version don't really know how to check
Screenshot if needed:

robiefish commented 7 years ago

The same issue for me. I usually only use favicons for websites and don't add any text to fit more bookmarks on my screen. Unfortunately it's not possible to add a bookmark with text only. It also seems that there is a limited number of icons that can be added to the bookmark bar if I just put in a space instead of text.

Another issues is that the favicon on some bookmarks doesn't update no matter how many times I refresh/restart.

mitchcurtis commented 6 years ago

+1, importing bookmarks from Chrome shows no favicons, and like @robiefish, I tend to rely solely on the favicon to identify the bookmark, rather than text.

luixxiul commented 6 years ago

Just in case did you switch settings on the preference page to display favicons? They are not displayed by default.

robiefish commented 6 years ago

Thanks for the suggestion Luixxiul, that's done the trick for me.

mitchcurtis commented 6 years ago

Thanks! Why is that disabled by default? :s

luixxiul commented 6 years ago

Why is that disabled by default?

I don't think there is a reason for that but I'm afraid it could be enabled by default once after https://github.com/brave/browser-laptop/issues/2697 is fixed.

cndouglas commented 6 years ago

Related: #2972 (Mixing favicon only and favicon + text)

This issue is a request for mixing favicon only and text only.

anornotan commented 5 years ago

My issue is not being able to simply use spaces in the text field to separate the bookmarks. It's supported in both Chromium and Firefox. In Preferences I have Text and Favicon on.

selection_001