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

Team member picture too small and broken social links in team member page #20

Open
arielj opened this issue Aug 24, 2021 · 2 comments
Open
Assignees
Labels
TODO Don't forget this! Type: UX/UI 💄 Frontend changes

Comments

@arielj
Copy link
Contributor

arielj commented Aug 24, 2021

The image looks too small, and the icons and handles for the different social links break into the next line (see the github or twitch icons are in one line and the handles are on the next

rubyforgood org_our-team_sean-marcia(iPhone X)

@meg-gutshall meg-gutshall added Help Wanted 🆘 Extra attention is needed Type: UX/UI 💄 Frontend changes labels Oct 27, 2021
@FionaDL
Copy link
Contributor

FionaDL commented Nov 3, 2021

@meg-gutshall I have played around with this a bit. A few questions:
For the photo would you prefer it to just be a bit larger, or to be full width?

127 0 0 1_4000_our-team_sean-marcia(iPhone X) (1)
127 0 0 1_4000_our-team_sean-marcia(iPhone X) (2)

For the icons what would you prefer? We could have them overflow the next lines, but then the "|" looks a little weird. We could take out the "|" all together, or maybe there is another option you prefer.

Screen Shot 2021-11-03 at 12 49 14 PM
Screen Shot 2021-11-03 at 12 49 01 PM
Screen Shot 2021-11-03 at 12 50 23 PM
Screen Shot 2021-11-03 at 12 50 39 PM

Thoughts?

@meg-gutshall
Copy link
Member

Hi @FionaDL! Thank you for taking a look at this and being patient waiting for my reply.

For the photo, I like the full-width look a lot, however, could you center the "Founder and CEO" title above the photo underneath the red line? I think that would look fantastic!

The links at the bottom is difficult... I'm wondering if there's a way to display the "|" pipes normally and then remove them at the mobile breakpoint because I really like the look of that last screenshot you took.

Also, Sean hard coded this into the page in our rush to get the MVP out. I'm trying to find some time to add my dynamic template back in which will take care of that trailing pipe at the end of the list of links. One step in the big process!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TODO Don't forget this! Type: UX/UI 💄 Frontend changes
Projects
None yet
Development

No branches or pull requests

3 participants