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

feat: add author icon to resource card (resolves #214) #216

Merged
merged 3 commits into from
Jan 31, 2020

Conversation

greatislander
Copy link
Collaborator

@greatislander greatislander commented Jan 30, 2020

Description

Adds author icon to resource card bylines.

Steps to test

Review resource card component: https://deploy-preview-216--pinecone.netlify.com

Additional information

Not applicable.

Related issues

@greatislander greatislander changed the title feat: add author icon to resource card (resolves feat: add author icon to resource card (resolves #214) Jan 30, 2020
@greatislander greatislander requested review from cherylhjli and jhung and removed request for cherylhjli January 30, 2020 21:33
@greatislander greatislander self-assigned this Jan 30, 2020
@greatislander greatislander added the enhancement New feature or request label Jan 30, 2020
@greatislander greatislander added this to the 1.0.0-alpha.10 milestone Jan 30, 2020
@netlify
Copy link

netlify bot commented Jan 30, 2020

Deploy preview for pinecone ready!

Built with commit 1f54887

https://deploy-preview-216--pinecone.netlify.com

@cherylhjli
Copy link

It looks great!
The spacing looks a bit funny between all the lines - can they be completely even between all metadata lines of text?

@greatislander
Copy link
Collaborator Author

The spacing is off, it turns out, because of the trick with the periods/middle dots as separators. I haven't figured out a solution yet.

@greatislander
Copy link
Collaborator Author

Fixed now!

@greatislander greatislander merged commit bb37e9a into dev Jan 31, 2020
@greatislander greatislander deleted the add/author-icon branch January 31, 2020 22:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Icon: author
3 participants