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

Update Card UI #183

Merged
merged 16 commits into from
Sep 8, 2023
Merged

Update Card UI #183

merged 16 commits into from
Sep 8, 2023

Conversation

hemarina
Copy link
Contributor

@hemarina hemarina commented Aug 11, 2023

No description provided.

@hemarina hemarina self-assigned this Aug 11, 2023
@hemarina
Copy link
Contributor Author

image

@hemarina
Copy link
Contributor Author

hemarina commented Aug 16, 2023

I'm using number of tags to determine whether we should use + x more tag. Once we review and apply changes on current card style. I'll add width map and use it to determine more tag. Or we could stay with what we have today.
image

@gkulin
Copy link
Collaborator

gkulin commented Aug 24, 2023

I will say I personally much prefer the +X more tag!

I did a quick comparison with Austin's design and I noticed a few things to update:
image

@Austinauth please feel free to correct me if I'm wrong about anything or add in any other feedback you have

@Austinauth
Copy link

@gkulin

I will say I personally much prefer the +X more tag!

I did a quick comparison with Austin's design and I noticed a few things to update: image

@Austinauth please feel free to correct me if I'm wrong about anything or add in any other feedback you have

This is great, thank you so much for red lining this out! Agree, I like the +x tag as well

@hemarina hemarina merged commit 69ecfbe into Azure:hemarina/newui Sep 8, 2023
1 check passed
@Austinauth
Copy link

Hey @hemarina going to start adding my feedback here. Things are getting really close. There are a couple adjustments I'd like to try that were not part of my initial mock-ups. Thank you for understanding!

CTA Card

image

Please reach out if you have any questions. We may also want to tweak the color of the text, but for now let's make these changes and see how it looks. Will follow up with feedback on the template cards tomorrow.

@Austinauth
Copy link

Template Card

Same as with the CTA Card above. I'm sure there will be some minor final adjustments we need to make after implementing these fixes. Please don't hesitate to reach out with questions. I will be out Friday thru Tuesday. No rush on my end.

Updated card design

image

Updated card design adjustments

Miscellaneous updates

image

Padding & Spacing

image

Copy behavior

image

Useful links

Here are some links you might find useful.

This was referenced Sep 19, 2023
@hemarina hemarina deleted the card1 branch October 20, 2023 20:39
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.

4 participants