Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Always generate manifest.json? #202

Open
brunocodutra opened this issue Mar 15, 2018 · 9 comments
Open

Always generate manifest.json? #202

brunocodutra opened this issue Mar 15, 2018 · 9 comments

Comments

@brunocodutra
Copy link
Collaborator

Now that browsers other than chrome are picking up on support for manifest.json does it make sense to have it always generated?

@alexander-akait
Copy link
Member

@brunocodutra better add option (by default in true) 👍

@musaffa
Copy link

musaffa commented Mar 24, 2018

I had to remove this package once I realized that it is also generating manifest.json and there is no way to disable it. I'm using another package for manifest management. I wish this package solely focused on favicons.

@alexander-akait
Copy link
Member

@musaffa we will have option for disable this 👍

@musaffa
Copy link

musaffa commented Jul 31, 2019

@evilebottnawi Is there any update about having an option to disable manifest generation?

@oller
Copy link

oller commented Oct 15, 2019

It seems manifest.json can support a lot of properties that aren't supported by favicons, i.e. related_applications, prefer_related_applications: etc.

Which as mentioned above is forcing users to use another package to handle the manifest generation, the loss being you're going to have to handle icons separately again.

If either this plugin could have a flag to disable manifest, or ideally take an options.seed object which it dumps into the manifest allowing support for all/any future manifest properties?

@slayerpart
Copy link

@evilebottnawi Any status update about optional manifest generation? This is really blocking us from using this module.

@alexander-akait
Copy link
Member

@slayerpart PR welcome

@johndeighan
Copy link

It appears that in the latest version, the manifest file's name has been changed. Maybe it's in response to the above complaints, but to me, it totally makes sense that this package generates a manifest file since a large part of the manifest file is the list of icons. I found it particularly annoying that the file extension was changed because now when I view the file in my editor, I don't get syntax lighting.

Luckily, I found a workaround that might also work for the above issues. It seems to be totally undocumented, but if in your config object you add the key files.android.manifestFileName and set it to the name you want the file to have, the resulting manifest file will have that name. Of course, that name is also used in the generated index.html file (which you can also rename).

For the maintainers, on the README page, there are comments that refer to the 'manifest.json' file, even though by default it's not named that. It might confuse some people (like me ;-)

@andy128k
Copy link
Collaborator

@johndeighan See #368

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants