[Merge] lp:~alexhenrie24/lubuntu-default-settings/fix-for-1232578 into lp:lubuntu-default-settings
Alex Henrie
alexhenrie24 at gmail.com
Mon Nov 4 22:21:45 UTC 2013
The standard is at http://standards.freedesktop.org/icon-naming-spec/icon-naming-spec-latest.html
It states: "An application must either use a generic application icon name provided by this specification, or install an icon named the same as the executable for running the application."
I emailed Rodney Dawes, one of the specification's authors, for clarification and got the following reply:
"The browser and mail icons were removed from the spec, because they are always branded applications, and not generic things. Only base generic things go into the spec. Browsers and mail apps should install branded icons in the
hicolor theme, named to match their executables, as stated in the spec.
"If lubuntu wants a generic script that launches the default browser, but always has a generic icon, then it should install it as a named icon to match the name of the wrapper script that runs the default app."
In this case, the executable is named x-www-browser, so according to the specification the icon must be named x-www-browser.png and NOT web-browser.png or internet-web-browser.png.
To fix the icon naming issue, web-browser.png has to be renamed to x-www-browser.png in the lubuntu-icon-theme package, internet-web-browser.png has to be renamed to x-www-browser.png in the lxde-icon-theme package, and lxde-x-www-browser.desktop has to be updated in the lxpanel package.
Because neither web-browser nor internet-web-browser is correct, and it will require the cooperation of many people to fix all the affected packages, would you be willing to put the icon naming issue aside for the moment and accept my patch to address the other issues?
--
https://code.launchpad.net/~alexhenrie24/lubuntu-default-settings/fix-for-1232578/+merge/193691
Your team Lubuntu Developers Team is subscribed to branch lp:lubuntu-default-settings.
More information about the Ubuntu-reviews
mailing list