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

fix: opportunty card responsiveness #6082

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
5 changes: 5 additions & 0 deletions src/reusecore/Layout/Row.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,11 @@ const RowWrapper = styled.div`
display: flex;
margin-left: -15px;
margin-right: -15px;
flex-wrap: wrap;
@media (max-width: 1440px) {
flex-wrap: wrap;
}

@media (max-width: 760px) {
flex-wrap: wrap;
}
Expand Down
5 changes: 3 additions & 2 deletions src/sections/Careers/Career-single/CareerSingle.style.js
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,10 @@ const CareerPageWrapper = styled.div`
border-bottom: 1px solid rgba(0, 0, 0, 0.2);
}

@media screen and (max-width: 767px) {
@media screen and (max-width: 1200px) {
.job-header-row {
flex-direction: column-reverse;
align-items: center;
}
}

Expand Down Expand Up @@ -74,7 +75,7 @@ const CareerPageWrapper = styled.div`
}
}

@media screen and (max-width: 767px) {
@media screen and (max-width: 1200px) {
.layer5-logo {
margin: 0 auto 1.5rem;
}
Expand Down
4 changes: 2 additions & 2 deletions src/sections/Careers/Career-single/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,8 @@ const CareerSingle = ({ data }) => {
<div className="single-post-wrapper">
<Container>
<Row className="job-header-row">
<Col $xs={12} $md={7} $xl={8} className="job-details">
<Row className="job-details-row">
<Col $xs={12} $md={12} $xl={8} className="job-details">
<Row>
<Col $xs={6} className="job-details-col">
<h5> <FaPlay /> Start Date</h5>
<p>{frontmatter.start_date}</p>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -115,14 +115,28 @@ p{
}

.full_time {
.opportunity-card {
flex-wrap: wrap;
.opportunity-card {
height: 12rem;
}
}

@media screen and (max-width: 991px) {
.opportunity-card {
height: 12.5rem;
width: 16rem;
Copy link
Contributor

Choose a reason for hiding this comment

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

Instead of having a fix width, it better practice to with max-width that would manage the card size even in below the specified screen sizes.

}
}

@media only screen and (max-width: 489px) {
.opportunity-card {
width: 100% !important;
}
}

@media screen and (max-width: 700px) {
.opportunity-card {
width: 23rem;
Copy link
Contributor

Choose a reason for hiding this comment

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

adding a perfect max-wdith would get adjusted when screen size is changed, it automatically get adjusted. You can set one by trying different values.

}
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,7 @@ export const AdventuresWrapper = styled.div`
margin-top: -36rem;
margin-left: 20rem;
}

display: flex;
}

Expand Down
1 change: 0 additions & 1 deletion src/sections/Community/Handbook/mentorships.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@ const ConductWrapper = styled.div`

padding: 0 5rem 3rem 20rem;
margin-top: -46.5rem;

@media screen and (min-width: 768px) and (max-height: 1145px) {
margin-top : -80vh

Expand Down
Loading