Christopher Aillon brought up a problem in the current icon code for PackageDB. It seems that mozilla is sensitive to the icon that is associated with the firefox web browser. So we shouldn't be showing icons other than the default or hicolor icon set for firefox or risk trademark repurcussions.
I've removed the other icons from the pkgdb for now but newer packages with icons for firefox that get imported into the database could potentially occur at any time.
Martin Bacovsky has a small patch for pkgdb that prioritises the themes that are used when choosing an icon. That should be enough to always choose a proper icon for displaying alongside firefox.
Could I get two +1s for applying this to the app servers?
=== modified file 'pkgdb/applications.py' --- pkgdb/applications.py 2010-03-15 16:26:59 +0000 +++ pkgdb/applications.py 2010-03-25 12:30:12 +0000 @@ -38,7 +38,7 @@
from pkgdb.model import Comment, Application, Icon, IconName, PackageBuild from pkgdb.model import Tag, Usage, ApplicationUsage, ApplicationTag -from pkgdb.model import MimeType +from pkgdb.model import MimeType, Theme from pkgdb.lib.utils import mod_grp from pkgdb import release, _ from pkgdb.lib.text_utils import excerpt @@ -531,15 +531,18 @@ def show(self, *app_name): app_name = '/'.join(app_name)
- # TODO: themes - icon_data = session.query(Icon.icon)\ - .join(Icon.name, IconName.applications)\ + icon_data = session.query(Theme.name, Icon.icon)\ + .join(Icon.name, IconName.applications, Icon.theme)\ .filter(Application.name==app_name)\ - .first() + .all() + if not icon_data: redirect('/static/images/noicon.png')
- return str(icon_data[0]) + icons = dict(icon_data) + + # icon theme priority: default > hicolor > whatever + return str(icons.get('default', None) or icons.get('hicolor', None) or icons[icons.keys()[0]])
-Toshio
-1 I think that is a bug in how mozilla as a organisation works and we should not do anything about it.
On Thursday 25 March 2010 11:41:10 am Toshio Kuratomi wrote:
Christopher Aillon brought up a problem in the current icon code for PackageDB. It seems that mozilla is sensitive to the icon that is associated with the firefox web browser. So we shouldn't be showing icons other than the default or hicolor icon set for firefox or risk trademark repurcussions.
I've removed the other icons from the pkgdb for now but newer packages with icons for firefox that get imported into the database could potentially occur at any time.
Martin Bacovsky has a small patch for pkgdb that prioritises the themes that are used when choosing an icon. That should be enough to always choose a proper icon for displaying alongside firefox.
Could I get two +1s for applying this to the app servers?
=== modified file 'pkgdb/applications.py' --- pkgdb/applications.py 2010-03-15 16:26:59 +0000 +++ pkgdb/applications.py 2010-03-25 12:30:12 +0000 @@ -38,7 +38,7 @@
from pkgdb.model import Comment, Application, Icon, IconName, PackageBuild from pkgdb.model import Tag, Usage, ApplicationUsage, ApplicationTag -from pkgdb.model import MimeType +from pkgdb.model import MimeType, Theme from pkgdb.lib.utils import mod_grp from pkgdb import release, _ from pkgdb.lib.text_utils import excerpt @@ -531,15 +531,18 @@ def show(self, *app_name): app_name = '/'.join(app_name)
# TODO: themes
icon_data = session.query(Icon.icon)\
.join(Icon.name, IconName.applications)\
icon_data = session.query(Theme.name, Icon.icon)\
.join(Icon.name, IconName.applications, Icon.theme)\ .filter(Application.name==app_name)\
.first()
.all()
if not icon_data: redirect('/static/images/noicon.png')
return str(icon_data[0])
icons = dict(icon_data)
# icon theme priority: default > hicolor > whatever
return str(icons.get('default', None) or icons.get('hicolor',
None) or icons[icons.keys()[0]])
-Toshio
On Thu, Mar 25, 2010 at 11:55:27AM -0500, Dennis Gilmore wrote:
-1 I think that is a bug in how mozilla as a organisation works and we should not do anything about it.
Well.. this is a legal issue; we could always stop distributing firefox. But that's really a board call. If we're going to continue distributing firefox then we need to play by the terms of the licenses we've been granted which means that our software has to take care not to violate them.
-Toshio
On Thu, 25 Mar 2010, Toshio Kuratomi wrote:
Christopher Aillon brought up a problem in the current icon code for PackageDB. It seems that mozilla is sensitive to the icon that is associated with the firefox web browser. So we shouldn't be showing icons other than the default or hicolor icon set for firefox or risk trademark repurcussions.
I've removed the other icons from the pkgdb for now but newer packages with icons for firefox that get imported into the database could potentially occur at any time.
Martin Bacovsky has a small patch for pkgdb that prioritises the themes that are used when choosing an icon. That should be enough to always choose a proper icon for displaying alongside firefox.
Could I get two +1s for applying this to the app servers?
=== modified file 'pkgdb/applications.py' --- pkgdb/applications.py 2010-03-15 16:26:59 +0000 +++ pkgdb/applications.py 2010-03-25 12:30:12 +0000 @@ -38,7 +38,7 @@
from pkgdb.model import Comment, Application, Icon, IconName, PackageBuild from pkgdb.model import Tag, Usage, ApplicationUsage, ApplicationTag -from pkgdb.model import MimeType +from pkgdb.model import MimeType, Theme from pkgdb.lib.utils import mod_grp from pkgdb import release, _ from pkgdb.lib.text_utils import excerpt @@ -531,15 +531,18 @@ def show(self, *app_name): app_name = '/'.join(app_name)
# TODO: themes
icon_data = session.query(Icon.icon)\
.join(Icon.name, IconName.applications)\
icon_data = session.query(Theme.name, Icon.icon)\
.join(Icon.name, IconName.applications, Icon.theme)\ .filter(Application.name==app_name)\
.first()
.all()
if not icon_data: redirect('/static/images/noicon.png')
return str(icon_data[0])
icons = dict(icon_data)
# icon theme priority: default > hicolor > whatever
return str(icons.get('default', None) or icons.get('hicolor', None) or icons[icons.keys()[0]])
-Toshio
I don't see this as harmful.
+1
-Mike
On Thu, Mar 25, 2010 at 04:19:17PM -0500, Mike McGrath wrote:
On Thu, 25 Mar 2010, Toshio Kuratomi wrote:
Christopher Aillon brought up a problem in the current icon code for PackageDB. It seems that mozilla is sensitive to the icon that is associated with the firefox web browser. So we shouldn't be showing icons other than the default or hicolor icon set for firefox or risk trademark repurcussions.
I've removed the other icons from the pkgdb for now but newer packages with icons for firefox that get imported into the database could potentially occur at any time.
Martin Bacovsky has a small patch for pkgdb that prioritises the themes that are used when choosing an icon. That should be enough to always choose a proper icon for displaying alongside firefox.
Could I get two +1s for applying this to the app servers?
I don't see this as harmful.
+1
-Mike
Got a second +1 from lmacken on IRC as well as discussion between dgilmore and caillon about the issues surrounding this. Change applied to app and bapp servers.
-Toshio
infrastructure@lists.fedoraproject.org