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: low contrast text and Istio box #4922

Merged
merged 7 commits into from
Oct 14, 2023
Merged

Conversation

SaxenaShiv
Copy link
Contributor

@SaxenaShiv SaxenaShiv commented Sep 16, 2023

Description
This PR fixes #4909
The text in the dark mode of the learning paths has been improved to address the visibility issue. Additionally, the border box surrounding the Istio logo has been eliminated.

Signed commits

  • Yes, I signed my commits.

@l5io
Copy link
Contributor

l5io commented Sep 16, 2023

🚀 Preview for commit 379e75a at: https://650575e7a043597a50680e2d--layer5.netlify.app

@l5io
Copy link
Contributor

l5io commented Sep 16, 2023

🚀 Preview for commit c50f5cb at: https://6505f867ef6fec3a97b2c454--layer5.netlify.app

@Ghat0tkach
Copy link
Member

hey @Shiv-2712
Let's discuss it on the websites call.
I have already added your agenda as part of today's meeting minutes

https://docs.google.com/document/d/1XczAHXVe2FIWPqiF57ospJ43zw5cZQ7ui8mn39v5EvA/edit#heading=h.lohhtewfwima

@@ -381,7 +381,7 @@ export const darktheme = {
grey313131ToYellowFFF7D8: "#313131",
grey323232ToGreenF3FFFD: "#323232",
grey3C3C3CToGreyF1F1F1: "#3C3C3C",
grey404040ToGreen3C494F: "#404040",
grey404040ToGreen3C494F: "#ffffff66",
Copy link
Member

Choose a reason for hiding this comment

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

Thanks, @Shiv-2712. Where else is this color used? Can you confirm that the use of this color in other locations on the site are not negatively affected?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Welcome! This specific color is exclusively employed in Learn-Components, specifically in the file Chapter Style (chapter.style.js). No other file utilizes this particular color variable.

@github-actions
Copy link

🚨 Alert! Git Police! We couldn’t help but notice that one or more of your commits is missing a sign-off. A what? A commit sign-off (your email address).

To amend the commits in this PR with your signoff using the instructions provided in the DCO check.

To configure your dev environment to automatically signoff on your commits in the future, see these instructions.


        Be sure to join the community, if you haven't yet and please leave a ⭐ star on the project 😄

@abhijeetgauravm
Copy link
Contributor

abhijeetgauravm commented Sep 24, 2023

Description The text in the dark mode of the learning paths has been improved to address the visibility issue. Additionally, the border box surrounding the Istio logo has been eliminated.

Signed commits

  • Yes, I signed my commits.

hey @Shiv-2712 it would be good if you just edit the description with like This PR fixes #issue_no

Signed-off-by: vishalvivekm <[email protected]>
Signed-off-by: Shiv Kumar Saxena <[email protected]>
@SaxenaShiv SaxenaShiv force-pushed the master branch 2 times, most recently from 1caaf3e to dfa5e59 Compare October 3, 2023 13:50
@SaxenaShiv SaxenaShiv force-pushed the master branch 5 times, most recently from 670b686 to a77d4d1 Compare October 3, 2023 13:50
vishalvivekm and others added 5 commits October 3, 2023 13:50
Signed-off-by: Vivek Vishal <[email protected]>
Signed-off-by: Shiv Kumar Saxena <[email protected]>
Signed-off-by: Shiv Kumar Saxena <[email protected]>
Signed-off-by: Shiv Kumar Saxena <[email protected]>
Signed-off-by: Shiv Kumar Saxena <[email protected]>
Signed-off-by: Shiv Kumar Saxena <[email protected]>
Signed-off-by: Shiv Kumar Saxena <[email protected]>
@l5io
Copy link
Contributor

l5io commented Oct 3, 2023

🚀 Preview for commit 87c7594 at: https://651c1ffb7992d33137a8eeaf--layer5.netlify.app

@Mohith234
Copy link
Member

@Shiv-2712 Update the PR description with correct mention of the issue

@abhijeetgauravm
Copy link
Contributor

Description This PR fixes #4922 The text in the dark mode of the learning paths has been improved to address the visibility issue. Additionally, the border box surrounding the Istio logo has been eliminated.

Signed commits

  • Yes, I signed my commits.

hey @Shiv-2712 Please mention the issue you are fixing. Here u are fixing #4909. So u should edit the PR description with this.

@abhijeetgauravm
Copy link
Contributor

Hey @Shiv-2712 Let's discuss this on website call today at 5:30 PM IST / 7 AM Central time. Please add this as an agenda item in the meeting minutes

Copy link
Member

@leecalcote leecalcote left a comment

Choose a reason for hiding this comment

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

Thanks!

@leecalcote leecalcote merged commit 4effe10 into layer5io:master Oct 14, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

Low Text contrast in dark mode for learning paths
7 participants