Missing icon in about:addons #16

Closed
pdehaan opened this Issue Dec 7, 2017 · 5 comments

Comments

Projects
None yet
2 participants
Contributor

pdehaan commented Dec 7, 2017

After installing temp addon in Firefox Nightly, I see the following in my about:addons page:

add-ons_manager

Contributor

pdehaan commented Dec 7, 2017

Ah, my local [generated] install.rdf looks like this, which may explain why I'm seeing missing name/icon:

    <em:name></em:name>
    <em:description>template shield study to serve a as base.  This one is about Toolbar Buttons</em:description>
    <em:creator>Gregg Lind &lt;glind@mozilla.com&gt;</em:creator>
    <em:iconURL>chrome://whiterose-experience-icons/content/icon.png</em:iconURL>

Note that I don't seem to have a "whiterose-experience-icons" directory anywhere, so that may be the cause of missing assets.

Contributor

pdehaan commented Dec 7, 2017

The <em:name> is blank because it's copying strings from package.json:

https://github.com/gregglind/addon-wr/blob/526535ea04ce79f59a79381ab71b8c9be61b9af8/package.json#L6-L18

https://github.com/gregglind/addon-wr/blob/526535ea04ce79f59a79381ab71b8c9be61b9af8/templates/install.rdf.mustache#L11

Note that the addon{} object doesn't contain a name property. Not sure if we can make it more generic name so it copies the package.json name.

Member

gregglind commented Dec 8, 2017

Ah! yes, that is a foo that I fixed other places. I will fix that.

Member

gregglind commented Dec 8, 2017

I solved name, but forgot icon. Will just rip it out probably!

@pdehaan pdehaan changed the title from Missing name+icon in about:addons to Missing icon in about:addons Dec 9, 2017

Member

gregglind commented Dec 12, 2017

Okay with this, as a wontfix.

@pdehaan pdehaan closed this Dec 12, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment