Skip to content

fix: teacher card styles - #122

Merged
Alaa2019-ml merged 2 commits into
developfrom
fix/teachers-cards
Feb 28, 2026
Merged

Alaa2019-ml merged 2 commits into
developfrom
fix/teachers-cards

Conversation

@Dmytro-Doronin

Copy link
Copy Markdown
Collaborator

Fixed teachers card responsiveness

Copilot AI review requested due to automatic review settings February 27, 2026 19:29
@HackYourFutures
HackYourFutures temporarily deployed to study-bridge-fix-teache-a31kro February 27, 2026 19:29 Inactive

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates UI responsiveness for the TeacherCard component (layout breakpoints, alignment, and section separation). The PR also changes the home Hero background image import, which appears unrelated to the stated goal.

Changes:

  • Adjust TeacherCard layout to switch to row layout at sm and tweak alignment/wrapping rules for inner content.
  • Replace the previous xl right border separation with an explicit divider element and adjust right-side pricing section sizing.
  • Update Hero to import a different background image asset.

Reviewed changes

Copilot reviewed 2 out of 4 changed files in this pull request and generated 4 comments.

File Description
client/src/components/teacherCard/teacherCard.tsx Responsiveness/layout tweaks (breakpoints, alignment, divider, typography).
client/src/components/homeSection/Hero/Hero.tsx Swaps hero background image import to a different asset.
Comments suppressed due to low confidence (1)

client/src/components/teacherCard/teacherCard.tsx:56

  • The avatar container className has redundant display utilities (block + md:block) and includes extra blank lines/whitespace in the template string. Cleaning this up will make future styling changes less error-prone and avoids accidental class mismatches during refactors.
            className="block
                md:block lg:hidden xl:block
                w-28 h-28
                md:w-37.5 md:h-37.5
                lg:w-43.25 lg:h-43.25
                xl:w-48 xl:h-48
                overflow-hidden rounded-[25px]

                             "

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread client/src/components/teacherCard/teacherCard.tsx
Comment thread client/src/components/homeSection/Hero/Hero.tsx Outdated
Comment thread client/src/components/teacherCard/teacherCard.tsx
Comment thread client/src/components/teacherCard/teacherCard.tsx Outdated
@HackYourFutures
HackYourFutures temporarily deployed to study-bridge-fix-teache-a31kro February 27, 2026 19:40 Inactive
@Alaa2019-ml
Alaa2019-ml merged commit cab4c68 into develop Feb 28, 2026
2 checks passed

This branch was previously deployed

1 inactive deployment
study-bridge-fix-teache-a31kro — bb6061ec Deployed Feb 27, 2026 by HackYourFutures
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