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

Pattern Submission: Team Social Cards #140

Closed
justintadlock opened this issue Jun 8, 2021 · 7 comments
Closed

Pattern Submission: Team Social Cards #140

justintadlock opened this issue Jun 8, 2021 · 7 comments
Labels

Comments

@justintadlock
Copy link

Team/Company member sections are pretty common. This pattern uses the Columns block to create a four-column group of team cards with avatars and social links.

Notes: Either Twenty Twenty-One or Gutenberg is overriding the social icon color on the front end. In the editor, they are white (as they should be), but on the front end, they are black.

I could also probably find a better default image, but the giraffe might just be fine too. Not sure about including photos of real people (even under CC0) and whether we need an additional model release for such things.


Pattern Title

Team Social Cards

Pattern Categories

Columns

Screenshots

screenshot

Image Credits

Public Domain / CC0 Images:

Block Markup

https://github.com/justintadlock/wporg-patterns/blob/master/columns-team-social-cards/pattern.html

@beafialho
Copy link
Collaborator

Love the giraffe! 😁 Not sure it fits this context in particular. A couple suggestions:

  • Have you considered naming each card with a real name? So, instead of "Person Name" use "John Doe" and instead of "Team Role", perhaps "Team Manager", to make the pattern look more real.
  • I also suggest replacing the giraffe with real people's photos, for the same reason mentioned above.

@kjellr
Copy link
Collaborator

kjellr commented Jun 9, 2021

Here's a suggested edit, taking @beafialho's suggestions into account:

  • I tried pulling some portraits from art, so that we don't have to worry about actual photos of people. I only found a decent set of three, so I trimmed this down into just three columns.
  • I changed the name to be more real.
  • I took a couple other liberties with the design: I changed the colors a bit to match the new images, moved the person's name and description out into a paragraph so we can style it a little more (in some themes, an image caption may enforce some weird styles for captions), and updated the social icon colors to match the text for a more considered appearance.

Here's how it turned out:

Screen Shot 2021-06-09 at 10 12 09 AM

Markup for all this is here:

https://gist.github.com/kjellr/d58e732719d5336fbddf81f1a3070505

Let me know what you folks think!


Image Credits:

Portrait of a Woman (1910) by Egon Schiele. Public Domain.
https://www.rawpixel.com/image/2681051/free-illustration-image-egon-schiele-portrait-schiele

Portrait of a Woman (1910) by Egon Schiele. Public Domain.
https://www.rawpixel.com/image/2680516/free-illustration-image-art-line-art-vintage

Portrait of a Woman (1910) by Egon Schiele. Public Domain.
https://www.rawpixel.com/image/2681051/free-illustration-image-egon-schiele-portrait-schiele

@justintadlock
Copy link
Author

I like the changes. Great work! Moving the text out into a paragraph also fixes the spacing issue I was having with using it in a caption.

@melchoyce
Copy link
Collaborator

This looks great, y'all nailed it. I think it's good to move over to the directory now 👍

@kjellr
Copy link
Collaborator

kjellr commented Jun 9, 2021

This one is live! Thanks for contributing! https://wordpress.org/patterns/pattern/team-social-cards/

It looks like there's a bug that's not displaying the social links, so I'll keep the issue open to track that.

Screen Shot 2021-06-09 at 11 23 24 AM

@kjellr kjellr added [Type] Bug Something isn't working and removed [Status] Needs Design Feedback labels Jun 9, 2021
@justintadlock
Copy link
Author

Also, I figured out the icon color issue in the OP.

When setting the Link Color for the Column block, WP/GB adds a <style> tag that targets .wp-elements-xxxx a { color: var( --wp--preset--color--black ) !important; }, which overrides social icon text colors.

It's noticable on the front-end but not the editor.

That might need to be addressed upstream.

@kjellr
Copy link
Collaborator

kjellr commented Jun 10, 2021

Since some of the display issues appear to span across multiple patterns, I've opened a single issue to track them. I'll close this issue as resolved.

#156

@kjellr kjellr closed this as completed Jun 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants