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

improve layout with icons #1

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

garethslinn
Copy link
Contributor

No description provided.

Copy link
Member

@getify getify left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add a commit to remove the .idea folder.

the icon images are a good idea. not sure on a few of them. but it's helpful to start thinking about some visual branding for them. can explore various icon sets.

which icon set did you get these from?

index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
@getify
Copy link
Member

getify commented Sep 18, 2024

LGTM. now I just want to think about which specific icons are best to represent these projects.

@garethslinn
Copy link
Contributor Author

Icons in HTML lists improve UX and accessibility by providing quick visual cues, enhancing appeal, and aiding in content scanning. I'm happy to change them or have them removed, just give me some clear instructions on what you would prefer.

BTW, I created those SVG's, here's a brief introduction to the design skills I have: https://www.gds-c.com/graphic-design/

@garethslinn
Copy link
Contributor Author

By the way, I’m open to feedback and more than willing to help you find the right branding for your products. I'll always explain the reasoning behind my ideas, but I'm not attached to them and am here to support your vision, if that makes sense.

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