Skip to content

Commit 30d9781

Browse files
committed
removed some comments
1 parent 4bc8544 commit 30d9781

File tree

2 files changed

+60
-69
lines changed

2 files changed

+60
-69
lines changed

pages/about.js

Lines changed: 58 additions & 67 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,6 @@ export default function AboutUs() {
88
return (
99
<div className='about-us'>
1010
<RevealContentContainer>
11-
{/* ================================FIRST ABOUT SECTION============== */}
12-
{/* <Container> */}
13-
{/* <Title title='Our background' /> */}
1411
<TwoColumn
1512
title='Our background'
1613
content={
@@ -31,75 +28,69 @@ export default function AboutUs() {
3128
bgColor={primaryAccent}
3229
customInnerClass='our-background'
3330
/>
34-
{/* </Container> */}
3531
</RevealContentContainer>
3632

3733
<RevealContentContainer>
38-
{/* <Container> */}
39-
{/* <Title title='Peer reviews' /> */}
40-
<TwoColumn
41-
title='Peer reviews'
42-
content={
43-
<div>
44-
Most of those new graduates used a version control feature, such
45-
as&nbsp;
46-
<a
47-
target='_blank'
48-
href='http://www.github.com'
49-
rel='noopener noreferrer'
50-
>
51-
GitHub
52-
</a>{' '}
53-
or&nbsp;
54-
<a
55-
target='_blank'
56-
href='http://www.bitbucket.com'
57-
rel='noopener noreferrer'
58-
>
59-
Bitbucket
60-
</a>
61-
, as a repository for their personal projects where they were
62-
the only participants. So, all that “branch out and PR review”
63-
enriching process was completely unknown to them.
64-
<br />
65-
<br />
66-
That happened to be a problem either for being considered for
67-
“junior” positions and after being hired because, in most cases,
68-
their team lead was a very, very busy senior developer…
69-
</div>
70-
}
71-
rowOrder='row-reverse'
72-
image='/images/svg/open-angle-bracket.svg'
73-
color={primary}
74-
bgColor={white}
75-
customInnerClass='about-content'
76-
/>
77-
<TwoColumn
78-
title='Version control'
79-
content={
80-
<div>
81-
Another problem found out is that collaborating with open-source
82-
projects, the way it is possible to gain some version control
83-
experience, can be quite intimidating since there isn’t a clear
84-
mentoring aspect.
85-
<br />
86-
<br />
87-
It can take years for someone to develop “a logic mindset” that
88-
allows solving a problem by using a programming language, so
89-
just going to an open-source project with a list of available
90-
tasks that don’t make any sense won’t cut it.
91-
</div>
92-
}
93-
image='/images/svg/close-angle-bracket.svg'
94-
color={primary}
95-
bgColor={white}
96-
customInnerClass='about-content'
97-
/>
98-
{/* </Container> */}
34+
<TwoColumn
35+
title='Peer reviews'
36+
content={
37+
<div>
38+
Most of those new graduates used a version control feature, such
39+
as&nbsp;
40+
<a
41+
target='_blank'
42+
href='http://www.github.com'
43+
rel='noopener noreferrer'
44+
>
45+
GitHub
46+
</a>{' '}
47+
or&nbsp;
48+
<a
49+
target='_blank'
50+
href='http://www.bitbucket.com'
51+
rel='noopener noreferrer'
52+
>
53+
Bitbucket
54+
</a>
55+
, as a repository for their personal projects where they were the
56+
only participants. So, all that “branch out and PR review”
57+
enriching process was completely unknown to them.
58+
<br />
59+
<br />
60+
That happened to be a problem either for being considered for
61+
“junior” positions and after being hired because, in most cases,
62+
their team lead was a very, very busy senior developer…
63+
</div>
64+
}
65+
rowOrder='row-reverse'
66+
image='/images/svg/open-angle-bracket.svg'
67+
color={primary}
68+
bgColor={white}
69+
customInnerClass='about-content'
70+
/>
71+
<TwoColumn
72+
title='Version control'
73+
content={
74+
<div>
75+
Another problem found out is that collaborating with open-source
76+
projects, the way it is possible to gain some version control
77+
experience, can be quite intimidating since there isn’t a clear
78+
mentoring aspect.
79+
<br />
80+
<br />
81+
It can take years for someone to develop “a logic mindset” that
82+
allows solving a problem by using a programming language, so just
83+
going to an open-source project with a list of available tasks
84+
that don’t make any sense won’t cut it.
85+
</div>
86+
}
87+
image='/images/svg/close-angle-bracket.svg'
88+
color={primary}
89+
bgColor={white}
90+
customInnerClass='about-content'
91+
/>
9992
</RevealContentContainer>
10093

101-
{/* ================================FIRST ABOUT SECTION============== */}
102-
10394
{/* <RevealContentContainer>
10495
<Container>
10596
<Title title="Our goals" />

styles/TwoColumn.module.scss

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,11 +58,11 @@
5858
}
5959

6060
.inner__content {
61-
margin-bottom: 5rem;
61+
// margin-bottom: 5rem;
6262

6363
@include desktop {
6464
flex-basis: 50%;
65-
margin-bottom: 0;
65+
// margin-bottom: 0;
6666
}
6767

6868
.title {

0 commit comments

Comments
 (0)