Skip to content

fix(careers): correct low contrast text color in dark mode#7750

Merged
saurabhraghuvanshii merged 8 commits into
layer5io:masterfrom
Katotodan:fix/low-contrast
May 24, 2026
Merged

fix(careers): correct low contrast text color in dark mode#7750
saurabhraghuvanshii merged 8 commits into
layer5io:masterfrom
Katotodan:fix/low-contrast

Conversation

@Katotodan
Copy link
Copy Markdown
Member

Description

Fix a visibility issue on the UI/UX Visual Designer Internship career page where text within certain paragraphs exhibits low contrast when dark mode is enabled.

Cause

The styling conflict originates between parent and child wrappers; the parent container enables dark mode configurations, but the child wrapper lacks an explicit text color assignment, causing it to fall back to an illegible default state.

Changes

  • Explicitly set the text color on the child layout wrapper to guarantee high visibility across both light and dark themes.
  • Core functional modifications are isolated to lines 71 through 75.
  • Remaining file modifications are strictly code formatting updates applied automatically by the linter.

Before

Screenshot 2026-05-22 at 11 27 28

Signed commits

  • Yes, I signed my commits.

Signed-off-by: DANIEL KATOTO <katotodan@gmail.com>
@Katotodan Katotodan self-assigned this May 22, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 22, 2026

PR Preview Action v1.6.3
Preview removed because the pull request was closed.
2026-05-24 19:24 UTC

@saurabhraghuvanshii
Copy link
Copy Markdown
Member

Texts looking off the color.

image

Katotodan and others added 6 commits May 22, 2026 12:09
Signed-off-by: DANIEL KATOTO <katotodan@gmail.com>
Signed-off-by: DANIEL KATOTO <katotodan@gmail.com>
Signed-off-by: DANIEL KATOTO <katotodan@gmail.com>
Signed-off-by: DANIEL KATOTO <katotodan@gmail.com>
@Bhumikagarggg
Copy link
Copy Markdown
Contributor

@Katotodan Thank you for your contribution! Let's discuss this during the website call tomorrow at 5:30 PM IST | 7 AM CST Add it as an agenda item to the meeting minutes, if you would 🙂

@saurabhraghuvanshii
Copy link
Copy Markdown
Member

Thanks @Katotodan lgtm!!

@saurabhraghuvanshii saurabhraghuvanshii merged commit 9ee64df into layer5io:master May 24, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants