Skip to content

Add 'desktop_browser' to the list of keys to ignore #24

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

Merged
merged 2 commits into from
Mar 31, 2021
Merged

Add 'desktop_browser' to the list of keys to ignore #24

merged 2 commits into from
Mar 31, 2021

Conversation

craighutch
Copy link

@craighutch craighutch commented Feb 3, 2021

'backgroundRadius' doesn't work when converting camelCase to underscore, so adding it's parent key of 'desktop_browser' to the ignore list to prevent the conversion.

the other camelCased keys here ('backgroundColor' and 'imageScale') seem to work either way, so no harm there.

Fixes RealFaviconGenerator/realfavicongenerator#449
Fixes RealFaviconGenerator/realfavicongenerator#458

Craig Hutchison added 2 commits February 2, 2021 20:57
'backgroundRadius' doesn't work when converting camelCase to underscore, so adding it's parent key of 'desktop_browser' to the ignore list to prevent the conversion
@phbernard phbernard merged commit 1dd3d91 into RealFaviconGenerator:master Mar 31, 2021
@phbernard
Copy link
Contributor

Thank you for your two PRs! Version 0.5.2 was published a minute ago.

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

Successfully merging this pull request may close these issues.

Desktop icons not generating Gulp 'generate-favicon' doesn't adhere to 'desktopBrowser' settings
2 participants