Skip to content

Commit 3719f91

Browse files
authored
Merge pull request #1908 from AtCoder-NoviSteps/#1906
🎨 Improve grade colors (#1906)
2 parents 7648a8e + 9f8819c commit 3719f91

File tree

4 files changed

+29
-27
lines changed

4 files changed

+29
-27
lines changed

src/lib/components/GradeLabel.svelte

+1-1
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@
4646
{#if taskGrade !== TaskGrade.PENDING}
4747
{grade}
4848
{:else}
49-
{'??'}
49+
{''}
5050
{/if}
5151
</div>
5252
</div>

src/lib/utils/task.ts

+4-4
Original file line numberDiff line numberDiff line change
@@ -265,17 +265,17 @@ export const getTaskGradeLabel = (taskGrade: TaskGrade | string) => {
265265
// https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Array/includes
266266
export const toChangeTextColorIfNeeds = (grade: string): string => {
267267
const gradeToWhiteText = [
268-
`${getTaskGradeLabel(TaskGrade.D1)}`,
268+
`${getTaskGradeLabel(TaskGrade.Q4)}`,
269+
`${getTaskGradeLabel(TaskGrade.Q3)}`,
270+
`${getTaskGradeLabel(TaskGrade.Q2)}`,
269271
`${getTaskGradeLabel(TaskGrade.D2)}`,
270-
`${getTaskGradeLabel(TaskGrade.D3)}`,
271272
`${getTaskGradeLabel(TaskGrade.D4)}`,
272273
`${getTaskGradeLabel(TaskGrade.D5)}`,
274+
`${getTaskGradeLabel(TaskGrade.D6)}`,
273275
];
274276

275277
if (gradeToWhiteText.includes(grade)) {
276278
return 'text-white';
277-
} else if (getTaskGradeLabel(grade) === TaskGrade.D6) {
278-
return 'text-atcoder-bronze';
279279
} else {
280280
return 'text-black';
281281
}

src/test/lib/utils/task_grade.test.ts

+8-6
Original file line numberDiff line numberDiff line change
@@ -475,14 +475,15 @@ describe('Task grade', () => {
475475
});
476476

477477
describe('to change text color if needs', () => {
478-
describe('when task grades from 11Q to 1Q are given ', () => {
478+
describe('when task grades with black text color are given ', () => {
479479
const testCases: TestCasesForTaskGrade = [
480480
{ taskGrade: TaskGrade.Q11, expected: 'text-black' },
481481
{ taskGrade: TaskGrade.Q10, expected: 'text-black' },
482482
{ taskGrade: TaskGrade.Q6, expected: 'text-black' },
483483
{ taskGrade: TaskGrade.Q5, expected: 'text-black' },
484-
{ taskGrade: TaskGrade.Q2, expected: 'text-black' },
485484
{ taskGrade: TaskGrade.Q1, expected: 'text-black' },
485+
{ taskGrade: TaskGrade.D1, expected: 'text-black' },
486+
{ taskGrade: TaskGrade.D3, expected: 'text-black' },
486487
];
487488

488489
runTests(
@@ -494,11 +495,12 @@ describe('Task grade', () => {
494495
);
495496
});
496497

497-
describe('when task grades from 1D to 5D are given', () => {
498+
describe('when task grades with white text color are given', () => {
498499
const testCases: TestCasesForTaskGrade = [
499-
{ taskGrade: TaskGrade.D1, expected: 'text-white' },
500+
{ taskGrade: TaskGrade.Q4, expected: 'text-white' },
501+
{ taskGrade: TaskGrade.Q3, expected: 'text-white' },
502+
{ taskGrade: TaskGrade.Q2, expected: 'text-white' },
500503
{ taskGrade: TaskGrade.D2, expected: 'text-white' },
501-
{ taskGrade: TaskGrade.D3, expected: 'text-white' },
502504
{ taskGrade: TaskGrade.D4, expected: 'text-white' },
503505
{ taskGrade: TaskGrade.D5, expected: 'text-white' },
504506
];
@@ -514,7 +516,7 @@ describe('Task grade', () => {
514516

515517
describe('when task grades 6D is given', () => {
516518
const testCases: TestCasesForTaskGrade = [
517-
{ taskGrade: TaskGrade.D6, expected: 'text-atcoder-bronze' },
519+
{ taskGrade: TaskGrade.D6, expected: 'text-white' },
518520
];
519521

520522
runTests(

tailwind.config.ts

+16-16
Original file line numberDiff line numberDiff line change
@@ -33,22 +33,22 @@ const config = {
3333
900: '#159957',
3434
},
3535
atcoder: {
36-
Q11: '#e4d3da',
37-
Q10: '#e3cad8',
38-
Q9: '#e4cde0',
39-
Q8: '#e4cbe8',
40-
Q7: '#ddbbe6',
41-
Q6: '#d9afe5',
42-
Q5: '#f7cc52',
43-
Q4: '#f1dd72',
44-
Q3: '#cde030',
45-
Q2: '#6dd452',
46-
Q1: '#41c7ad',
47-
D1: '#1c85b6',
48-
D2: '#7f36be',
49-
D3: '#e68e2e',
50-
D4: '#e36223',
51-
D5: '#e60d00',
36+
Q11: '#e4e3e3',
37+
Q10: '#e4dfdc',
38+
Q9: '#dde3dc',
39+
Q8: '#dde7e7',
40+
Q7: '#dddceb',
41+
Q6: '#e8e7dc',
42+
Q5: '#dda0dd',
43+
Q4: '#da70d6',
44+
Q3: '#9370DB',
45+
Q2: '#7f6000',
46+
Q1: '#01fb02',
47+
D1: '#72C6ef',
48+
D2: '#002eff',
49+
D3: '#ffff02',
50+
D4: '#ff9900',
51+
D5: '#ff1000',
5252
D6: '#432414',
5353
gray: '',
5454
brown: '',

0 commit comments

Comments
 (0)