Skip to content

Commit

Permalink
feat: improve accessibility of text elements
Browse files Browse the repository at this point in the history
- Add light blue color.
- Increase contrast of text elements.
  • Loading branch information
Splode committed May 25, 2020
1 parent 5bd03d6 commit 20a56b7
Show file tree
Hide file tree
Showing 5 changed files with 24 additions and 26 deletions.
4 changes: 2 additions & 2 deletions src/renderer/assets/stylesheets/_base.scss
Original file line number Diff line number Diff line change
Expand Up @@ -54,12 +54,12 @@ p {
}

.TextButton {
color: $colorBlueGrey;
color: $colorBlueGrey--lighter;
font-size: 14px;
letter-spacing: 0.05em;
margin-top: 12px;
transition: $transitionDefault;
&:hover {
color: $colorRed;
color: $colorGreen;
}
}
2 changes: 2 additions & 0 deletions src/renderer/assets/stylesheets/_variables.scss
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,8 @@

$colorBlue: #0bbddb;
$colorBlueGrey: #858c99;
$colorBlueGrey--lighter: #c0c9da;
$colorBlueWhite: #dbe1ef;
$colorGreen: #05ec8c;
$colorLightNavy: #3d4457;
$colorNavy: #2f384b;
Expand Down
7 changes: 2 additions & 5 deletions src/renderer/components/drawer/Drawer-settings.vue
Original file line number Diff line number Diff line change
Expand Up @@ -25,10 +25,7 @@
:class="notifications ? 'is-active' : 'is-inactive'"
></div>
</div>
<div
class="Setting-wrapper"
v-if="os === 'win32'"
>
<div class="Setting-wrapper" v-if="os === 'win32'">
<p class="Setting-title">Minimize to Tray</p>
<div
class="Checkbox"
Expand Down Expand Up @@ -141,7 +138,7 @@ export default {
}
.Setting-title {
color: $colorBlueGrey;
color: $colorBlueGrey--lighter;
font-size: 14px;
letter-spacing: 0.05em;
}
Expand Down
35 changes: 17 additions & 18 deletions src/renderer/components/drawer/Drawer-timer.vue
Original file line number Diff line number Diff line change
Expand Up @@ -13,12 +13,11 @@
class="Slider Slider--red"
v-model.number="localTimeWork"
@change="setTimeWork($event, 'work')"
>
/>
<div
class="Slider-bar Slider-bar--red"
:style="{ width: calcPercentage(localTimeWork, maxTime) + '%' }"
>
</div>
></div>
</div>
</div>

Expand All @@ -34,12 +33,11 @@
class="Slider Slider--green"
v-model.number="localTimeShortBreak"
@change="setTimeShortBreak($event, 'short-break')"
>
/>
<div
class="Slider-bar Slider-bar--green"
:style="{ width: calcPercentage(localTimeShortBreak, maxTime) + '%' }"
>
</div>
></div>
</div>
</div>

Expand All @@ -55,12 +53,11 @@
class="Slider Slider--blue"
v-model.number="localTimeLongBreak"
@change="setTimeLongBreak($event, 'long-break')"
>
/>
<div
class="Slider-bar Slider-bar--blue"
:style="{ width: calcPercentage(localTimeLongBreak, maxTime) + '%' }"
>
</div>
></div>
</div>
</div>

Expand All @@ -76,20 +73,18 @@
class="Slider"
v-model.number="localWorkRounds"
@change="setWorkRounds"
>
/>
<div
class="Slider-bar Slider-bar--blueGrey"
:style="{ width: calcRoundPercentage(localWorkRounds, maxRounds) + '%' }"
>
</div>
:style="{
width: calcRoundPercentage(localWorkRounds, maxRounds) + '%'
}"
></div>
</div>
</div>

<div class="Setting-wrapper">
<p
class="TextButton"
@click="resetDefaults"
>Reset Defaults</p>
<p class="TextButton" @click="resetDefaults">Reset Defaults</p>
</div>
</div>
</template>
Expand Down Expand Up @@ -211,7 +206,7 @@ export default {
}
.Setting-title {
color: $colorBlueGrey;
color: $colorBlueWhite;
font-size: 14px;
letter-spacing: 0.05em;
margin-bottom: 8px;
Expand All @@ -225,4 +220,8 @@ export default {
font-size: 12px;
padding: 2px 6px;
}
.TextButton {
color: $colorBlueWhite;
}
</style>
2 changes: 1 addition & 1 deletion src/renderer/components/timer/Timer-footer.vue
Original file line number Diff line number Diff line change
Expand Up @@ -249,7 +249,7 @@ export default {
text-align: center;
.Total-rounds {
color: #858c99;
color: $colorBlueGrey--lighter;
font-size: 0.7rem;
}
}
Expand Down

0 comments on commit 20a56b7

Please sign in to comment.