Created attachment 655767 [details] Patch I added some nice high res favicons to the minimart. They look a little.... messed up when drawn as launcher icons because they're bigger than what my phone wants. We should either a.) Just draw these the same as we do for webapps (i.e. scale the icon to the right size and don't add a background or b.) Draw them to fit inside the box I'm going to take b here so that there's a difference between webapps and favicons, BUT... I'm not sure that is good enough of a reason. Opinion?
Created attachment 655768 [details] [diff] [review] Patch
Comment on attachment 655768 [details] [diff] [review] Patch Help me understand this first. When you say "favicon" and "shortcut", does that mean you are just adding a shortcut to the homescreen? or are you installing a webapp?
Heh. We need to refine our language here, but I'm talking about adding a shortcut to the homescreen. i.e. cases where we're drawing our special rectangle behind the icon.
OK. Shortcuts, not WebApps, should always use the "background", so using approach B should be the right way.
I moved this patch to bug 783921.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 783921
Comment on attachment 655768 [details] [diff] [review] Patch I fixed this in bug 783921
You need to log in before you can comment on or make changes to this bug.