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

Support to show custom icons #68

Closed
TobiasDev opened this issue May 13, 2020 · 7 comments
Closed

Support to show custom icons #68

TobiasDev opened this issue May 13, 2020 · 7 comments

Comments

@TobiasDev
Copy link

In KeePassXC (which is what I currently use on Windows and Linux until Authpass arrives since the design is terrible), you can add custom icons for all your entires (or download fav-icons). When I open the database on my phone in AuthPass I get a key icon. Are you planning on adding support for custom icons in AuthPass at some point?

grafik

@hpoul
Copy link
Collaborator

hpoul commented May 13, 2020

Right now only predefined icons are supported. tbh I didn't think anyone would use custom icons :-)
Shouldn't be too hard to support them though..

favicons for each service are a bit different.. I guess it would make sense to download them on-demand, rather than storing them in the kdbx file. (I'm trying to keep those files as small as possible).

btw. do you enter a nice "Website" field for every service? I personally just enter the rough domain name as "title" and never mind the "Website" field ;-)

@TobiasDev
Copy link
Author

Of-course I use the website field. :D Otherwise the auto-fill on the Firefox plugin don't work. I'm lazy, so I want it to be as easy and fast as possible to log into websites. :D Though not lazy enough to use something like Bitwarden since I still want to have my "own" password file.

Also, I believe that favicons are downloaded and added to the file in KeePassXC. Then they act like any other custom icon I add myself. :) I mostly use Custom Icons since A. They are ugly as hell in KeePassXC, I love the software and such, but that UI is very Windows XP and B. It's a lot easier on a quick glance to see what is to what. Sometimes I have several logins for the same webpage. E.G. Personal usage, company usages, etc. Then I use a custom icon with different BG colors to quickly see which one I need. :)

@TobiasDev
Copy link
Author

I should probably mention that I care a lot about the UI of my apps. I have changed from one app to something not as good JUST to get better UI. :D I only use apps with terrible UI if I really can't find another app that is even close to the functionallity. Like in the case with KeePassXC currently on Windows.

hpoul added a commit that referenced this issue Aug 13, 2020
@hpoul
Copy link
Collaborator

hpoul commented Aug 13, 2020

fyi, I have now implemented showing of custom icons (not changing).. but i'm not quite sure this improves the UI, because the favicons added by keepassxc are of pretty bad quality:

Screenshot 2020-08-13 at 13 12 32

in the above screenshot they are scaled up, if i don't scale them up, they don't look much better:

Screenshot 2020-08-13 at 13 13 29

I did not yet add support for changing custom icons. I honestly think it would be nicer to fetch the icons from the internet in a better quality. But this would involve some magic if the user does not have complete, valid URLs in their 'Website'(URL) field.

@TobiasDev
Copy link
Author

Hmm, there you are correct. :( I thought the grabbed them in higher quality, but apperently not. :(

@hpoul
Copy link
Collaborator

hpoul commented Aug 15, 2020

I've now played around a bit to dynamically load the icons.. I actually quite like it.. but I still have to solve a few usability problems.. like when a user still wants to change the icon.. or when I was not able to load the correct icon..

Screenshot 2020-08-15 at 12 29 00

I'm also not yet completely sure about the privacy/security implications. The 'Website' domains (not the full URL, just scheme + host) are sent to my server and i use some magic to find a good icon/logo.

I almost think the most sensible thing would be to make it a kdbx-file based option.. so when opening an existing kdbx file, it is turned off by default, but for new users I can add a hint in the 'Website' field.. but the most private thing this would leak are internal host names if someone adds a private intranet domain.. i hope noone uses that field to store passwords.. even then it would be strange if i'm able to parse the password as a URL :-) (unparsable strings aren't sent to the server either)

@hpoul
Copy link
Collaborator

hpoul commented Sep 26, 2020

I think displaying of custom icons is finished. I've separated the open issues into:

  1. Adding/selectig custom icos: Support changing to custom icons (for groups and entries) #157
  2. Rollout for dynamic website icons Rollout for dynamic website icons #158

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