| Summary: | Use "folder" icon for inode/directory as per icon naming spec | ||
|---|---|---|---|
| Product: | shared-mime-info | Reporter: | Sebastian Keller <sebastian-keller> |
| Component: | freedesktop.org.xml | Assignee: | Shared Mime Info group <shared_mime_info> |
| Status: | RESOLVED FIXED | QA Contact: | |
| Severity: | normal | ||
| Priority: | medium | ||
| Version: | unspecified | ||
| Hardware: | Other | ||
| OS: | All | ||
| Whiteboard: | |||
| i915 platform: | i915 features: | ||
| Attachments: | Use "folder" icon for inode/directory as per icon naming spec | ||
|
Description
Sebastian Keller
2015-02-14 23:12:47 UTC
Created attachment 113498 [details] [review] Use "folder" icon for inode/directory as per icon naming spec Comment on attachment 113498 [details] [review] Use "folder" icon for inode/directory as per icon naming spec Folder isn't one of the available generic-icons in the "Standard MIME Type Icons" table[1], it's a place icon. It's also not listed in the DTD at the top of the xml file. I don't understand which bug you're trying to fix, but it seems that it should be handled in GIO or in Adwaita instead. [1]: Table 10 at http://standards.freedesktop.org/icon-naming-spec/icon-naming-spec-latest.html I think I might be interpreting the specs wrong then. It seems to me as if the generic icon handling is not limited to items from the "MIME Types" table but could also include items from other tables (such as the "Places" table) as well. And the comment in the DTD [0] seems to apply here as well: "folder" can't be "computed" from the mime type "inode/directory".
You're right about "folder" not being listed in the DTD, I should have added it in the patch, but if this patch is the wrong approach anyway, that doesn't really matter anymore.
The bug I'm trying to fix is that g_content_type_get_icon("inode/directory") returns a "inode-directory" icon which does not exist in Adwaita. A fix in gio would be to handle this special case explicitly with a strcmp on the mime type rather than relying on the generic icon mechanism but that seemed a bit ugly to me. Adwaita itself seems to be in line with the icon naming spec and adding the "inode-directory" icon back would only be working around bugs in other parts of the stack.
Feel free to close this bug if I'm just reading the specs wrong. I'll try to fix it in gio then.
[0] http://cgit.freedesktop.org/xdg/shared-mime-info/tree/freedesktop.org.xml.in#n21
commit 052b7d7d73207bea8f623d0bdb108082e956d6bb Author: Sebastian Keller <sebastian-keller@gmx.de> Date: Sat Feb 14 23:55:30 2015 +0100 Use "folder" generic-icon for inode/directory As mentioned in the "Places" table of the icon naming spec. This avoids having to sprinkle special cases in GIO for avoiding the old x-inode-directory icon name. https://bugs.freedesktop.org/show_bug.cgi?id=89150 |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.