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

Create OurTeam component #136

Closed
wants to merge 6 commits into from
Closed

Conversation

samiba6
Copy link
Contributor

@samiba6 samiba6 commented Nov 22, 2023

Description
This pull request implements the OurTeam component, enhancing the user interface of the Student's Store by introducing an engaging presentation of team members. The component displays team member details upon hover, showcasing their names, roles, and associated social links. This creates an interactive experience for users exploring our team.

Changes Made
Created the OurTeam component: OurTeam.js.
Integrated the OurTeam component into the About us page.
Added hover animations for displaying member details and social links.
Ensured responsiveness across various screen sizes.
Screenshots
For Large Screens

image

For Small screens
image

Related Issue
Resolves #130

Related Issue

@samiba6 samiba6 added the component component label Nov 22, 2023
@samiba6 samiba6 linked an issue Nov 22, 2023 that may be closed by this pull request
Copy link

vercel bot commented Nov 22, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
student-store ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 25, 2023 3:38pm

Copy link
Collaborator

@KatiaGhezali KatiaGhezali left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your effort Sami, however I have noticed some things that need to be fixed:

  • The hover effect only works on large screens. In md and sm screens, you cannot see the team member information.
  • There is still a slight delay when the cards open(Almost hard to notice).
  • The color is beautiful, however not vert compatible with the palette we already have.

Please try to see to these little details, and again thank you for your work.

@Ben-Tewfik
Copy link
Contributor

In #118 I worked on this component and push it in this #165 pull request waiting for the review

@Ben-Tewfik Ben-Tewfik closed this Nov 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component component
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create Our Team component
3 participants