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

Please consider adding an icon for .kdbx files #808

Closed
4jNsY6fCVqZv opened this issue Aug 1, 2019 · 16 comments · Fixed by elementary/files#1006
Closed

Please consider adding an icon for .kdbx files #808

4jNsY6fCVqZv opened this issue Aug 1, 2019 · 16 comments · Fixed by elementary/files#1006

Comments

@4jNsY6fCVqZv
Copy link

4jNsY6fCVqZv commented Aug 1, 2019

To my understanding and knowledge this is a very common file format for password manager databases. Currently Files only shows a sad and not explaining empty grey-white icon. Do you have any idea how to improve this?


Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@4jNsY6fCVqZv 4jNsY6fCVqZv changed the title Consider adding an icon for .kdbx files Please consider adding an icon for .kdbx files Aug 1, 2019
@jeremypw
Copy link

jeremypw commented Aug 1, 2019

To fix this there needs to be a suitable file in /usr/share/mime/application defining which icon to use for this mime-type. Its not really a Files problem I am afraid (unless you mean that the fallback icon should be improved). I would have thought that an application that uses this file type would insert a suitable file?

@jeremypw
Copy link

jeremypw commented Aug 1, 2019

See "/usr/share/mime/applications/x-dbf.xml" for an example.

@4jNsY6fCVqZv
Copy link
Author

4jNsY6fCVqZv commented Aug 1, 2019

I know that's not a Files problem. That's why I created the issue at elementary/icons. I can also create it as an issue at KeePassXC. More sustainable would probably be the solution with a super nice fallback icon in elementary OS itself, right? Otherwise numerous apps would have to change something. How does elementary OS handle all the other mime types?

@4jNsY6fCVqZv
Copy link
Author

4jNsY6fCVqZv commented Aug 1, 2019

I would have thought that an application that uses this file type would insert a suitable file?

I'm afraid it doesn't look that way :(

@jeremypw
Copy link

jeremypw commented Aug 2, 2019

@4jNsY6fCVqZv Sorry, for some reason I thought I was on the Files repo :-(.

It seems that KeePass does create the necessary mime file and specifies the icon "application-x-keepassx" and icons of that name are installed in the "hicolor" theme. But Files (nor another file manager) is using it. I've tried running update-mime-database, gtk-update-icon-cache and update-desktop-database to no avail even after loggiing out and in.

I notice another 3rd party program has the same problem - and it used to work with mimetype icons in the hicolor theme.

I tried manually copying the keepass icons into the corresponding elementary theme folders and running gtk-update-icon-cache /usr/share/icons/elementary and then it works OK.

@jeremypw
Copy link

jeremypw commented Aug 2, 2019

I am not an expert on icon themes, but the problem may be that elementary is not falling back to use the hicolor theme any more?

@jeremypw
Copy link

jeremypw commented Aug 2, 2019

Looking at Files code, it seems that the icon is falling back to a generic icon within the elementary theme before looking in the hicolor theme. This sees to be related to the way Gtk.IconTheme.choose_icon_for_scale () works (now). I can push a PR forcing a look for the specified icon in the hicolor theme before falling back.

@jeremypw
Copy link

jeremypw commented Aug 2, 2019

As it turns out this was a Files issue - probably introduced when some C code handling icon lookup and caching was ported to Vala :-(

@4jNsY6fCVqZv
Copy link
Author

Thanks for taking the time to test this out! Do you think https://github.com/elementary/icons/issues/809 and #789
could be related to this issue?

@jeremypw
Copy link

jeremypw commented Aug 2, 2019

I think #789 is probably due to the same bug in Files. There is a different PR submitted that fixes it by symlinking to the hicolor icons but that would have to be done for each missing icon. I'll check whether my PR fixes it as well.

@4jNsY6fCVqZv
Copy link
Author

Thank you, that would be wonderful, if your PR could fix several problems!

@4jNsY6fCVqZv
Copy link
Author

@jeremypw Don't you think elementary/os-patches#63 could be related here?

@jeremypw
Copy link

jeremypw commented Aug 7, 2019

@4jNsY6fCVqZv Maybe but not directly. The icon provided by the app should be good enough.

@4jNsY6fCVqZv
Copy link
Author

@jeremypw I wonder whether the OpenConnect VPN client is really a separate app in this case, or just an additional plugin for the network manager?
OpenVPN and PPTP are the default options in elementary OS. So these are two independent apps, each with its own icons and dialogs?!

@jeremypw
Copy link

jeremypw commented Aug 9, 2019

That is best discussed on the related issue reports.

@danirabbit
Copy link
Member

Gonna close this since it sounds like this was a Files bug

@danirabbit danirabbit closed this as not planned Won't fix, can't repro, duplicate, stale Feb 8, 2024
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

Successfully merging a pull request may close this issue.

3 participants