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

Example of how to change size #1042

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

boukeversteegh
Copy link

The word 'size' is not mentioned anywhere in the readme or on the website, giving the impression that changing size is completely impossible.

This low-hanging fruit example could help people find the solution more easily.

@boukeversteegh
Copy link
Author

Solves #1041

@codecov
Copy link

codecov bot commented Feb 19, 2021

Codecov Report

Merging #1042 (0e637ba) into master (a718a7e) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master     #1042   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            5         5           
  Lines           34        34           
  Branches         3         3           
=========================================
  Hits            34        34           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a718a7e...f07df61. Read the comment docs.

@locness3
Copy link

This is an SVG iconset, you're expected to know how svg tags work (IMO).

@boukeversteegh
Copy link
Author

From my consumer perspective, how the icons are rendered and details of SVG are internals to the product. The product is that I can easily display an icon in an html file, by using only CSS <i class="feather" data-feather="user"/>.

So I find it strange to be expected to know how SVG tags work, because even if I do know, it also implies that I should know that styling of the original <i> is transferred to the generated SCG, which is another under-the-hood aspect.

Font-awesome is explicit about how it works, has a documentation page about sizing, and even provides helper classes to scale icons. That's why it's very user friendly and so popular. I hope feather can be as user friendly and popular.

@locness3
Copy link

Feather is not really maintained anymore - you may want to come over at @lucide-icons

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.

2 participants