-
-
Notifications
You must be signed in to change notification settings - Fork 28
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
Card UX Improvement #169
Comments
/assign |
@arkid15r what your view on this 🤔. any suggestion would be helpful |
We show all social links at the moment. If a project has a lot of them on the same platform that causes multiple icons of the same platform. I'm open to redesigning that based on UI/UX best practices. BTW, check it on the new frontend if you used old one for this issue research. Thank you! |
sorry for the late response, so here is some insight and changes I am planning on:
what is your thought on it ? @arkid15r |
@TenzDelek sorry I couldn't look into this today. I'll do it tomorrow first thing. |
sure @arkid15r |
@TenzDelek can I suggest the design for the messy social links. |
Hmm, thank you for the suggestion, I got your point but it would be still inconvenient for the users. Don't you think so. For example, you see a bunch of same icon in row without and text info, what will be your first thought. Wouldn't it be like "oh it looks like a bug or duplication ". If we are going to implement it, let's put helper text beside it. What my thought process as a designer was whether it is necessary? Because we don't want more harm than good being in the perspective of the end users. What I suggested is that we can put the overall link of the project (each for slack channel, twitter and GitHub) ,not the individuals. Which can solve the issue of duplicate. But still @harsh3dev idea also seems to be a choice. Let's wait for what @arkid15r has to say on this |
I don't have a strong opinion on this and kind of like it as is (the grayscale was intentional). Let's see the colorful version
Yes, that can be removed
idea 1: show only 1 icon per social (just to indicate what project has in general) and show a list of the links in a separate area.
I can only help with OWASP branding colors (see the last slide) |
Thank you for the input@arkid15r the idea 2 seems more intuitive. Will work on this soon |
could be useful -- https://owasp.org/www-policy/operational/branding |
appreciate it 👍, will finalize this issue this weekend |
I would like to work on this issue. Could you please assign it to me? I’ll ensure the card layout is improved and the social links are more functional and less redundant. |
Hey @vivekbisen04 , thank you for your interest, but I am currently working on this. The pr will be ready soon. It is taking some additional time because the card component is used across all pages , each with different layout |
@vivekbisen04 you can still contribute as a reviewer or join our project contributors team and pick any available tasks. |
sorry for the delay, was bit busy during the holiday season, will send the design tonight |
Sharing the figma file https://www.figma.com/design/NGGoRrzevMKSAESwqheSDR/owasp?node-id=0-1&t=FnuGIisWrlFoeZW1-1 , also Commute the same in slack. |
Describe the bug
Currently the socials links for the cards seems more like an duplication rather than a feature.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
An improvement for the card layout should be maintain.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: