Skip to content

Commit 288d611

Browse files
authored
Merge pull request #518 from topcoder-platform/TCA-1018-fix
TCA-1018 better modal view
2 parents ee7bb81 + 0036629 commit 288d611

File tree

2 files changed

+27
-8
lines changed

2 files changed

+27
-8
lines changed

src-ts/tools/learn/tca-certificate/validate-certificate/ValidateTCACertificate.module.scss

Lines changed: 25 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,7 @@
5858
>img {
5959
border: 3px solid $tc-white;
6060
border-radius: 50%;
61+
aspect-ratio: 1;
6162
}
6263

6364
.memberInfo {
@@ -149,6 +150,10 @@
149150
margin-top: $space-xxl;
150151

151152
@include ltelg {
153+
grid-template-columns: 1fr 1fr;
154+
}
155+
156+
@include ltesm {
152157
grid-template-columns: 1fr;
153158
}
154159

@@ -171,8 +176,9 @@
171176

172177
.modalView {
173178
.hero {
174-
padding-top: $space-lg;
175-
max-height: 360px;
179+
padding-top: $space-mx;
180+
max-height: 440px;
181+
min-height: 380px;
176182

177183
@include ltesm {
178184
max-height: none;
@@ -188,18 +194,31 @@
188194
}
189195

190196
.heroLeft {
191-
transform: scale(0.67);
192-
transform-origin: 0 0;
197+
.member {
198+
margin-bottom: $space-xxxxl;
199+
200+
>svg,
201+
>img {
202+
max-width: 88px;
203+
max-height: 88px;
204+
}
205+
}
193206

194-
@include ltesm {
195-
transform: none;
207+
.certTitle {
208+
font-size: 40px;
209+
line-height: 40px;
196210
}
197211
}
198212
}
199213
}
200214

201215
.contentOuter {
202216
max-width: none;
217+
padding-right: $space-mx;
218+
219+
@include ltesm {
220+
padding-right: $space-lg;
221+
}
203222

204223
>div {
205224
max-width: none !important;

src-ts/tools/learn/tca-certificate/validate-certificate/ValidateTCACertificate.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -144,8 +144,8 @@ const ValidateTCACertificate: FC<{}> = () => {
144144
</div>
145145
</div>
146146
</div>
147-
<p className='body-large'>
148-
{profile.handle}
147+
<p className={classNames(isModalView ? 'body-medium' : 'body-large')}>
148+
{enrollment?.userName}
149149
{' '}
150150
has successfully met the requirements of the certification
151151
</p>

0 commit comments

Comments
 (0)