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

Bower - Added fonts to "main" files #9964

Merged
merged 1 commit into from
Aug 24, 2013
Merged

Bower - Added fonts to "main" files #9964

merged 1 commit into from
Aug 24, 2013

Conversation

thomaswelton
Copy link
Contributor

When using bower the "main" property is used to specify the files that should be consumed by applications including the package as a dependency.
Without including the fonts in this array the glyphicons do not work correctly as the relative font paths do not work after the "main" files are copied into my projects public dir using tools like https://github.com/yatskevich/grunt-bower-task

For now I have manually copied the fonts dir in the location needed for my project. But by accepting this pull request this will not be necessary for me or for other people that use bower.

When using bower the "main" property is used to specify the files that should be consumed by applications including the package as a dependency.
Without including the fonts in this array the glyphicons do not work correctly as the relative font paths do not work after the "main" files are copied into my projects public dir using tools like https://github.com/yatskevich/grunt-bower-task

For now I have manually copied the fonts dir in the location needed for my project. But by accepting this pull request this will not be necessary for me or for other people that use bower.
@thomaswelton
Copy link
Contributor Author

@BrenoC updated to use "./dist/fonts/*" as per your comment.
I amended this change to my last commit to try to clean it up. But look like that deleted you comment? But I wasn't aware that'd happen

@klzns
Copy link
Contributor

klzns commented Aug 21, 2013

@thomaswelton Oh, that's fine, I guess that's git's proper behavior. Thanks for your concern :)

mdo added a commit that referenced this pull request Aug 24, 2013
Bower - Added fonts to "main" files
@mdo mdo merged commit a6a97e1 into twbs:master Aug 24, 2013
@mdo mdo mentioned this pull request Aug 24, 2013
@thomaswelton thomaswelton deleted the patch-1 branch June 10, 2014 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants