Skip to content

Fix/highcontrast transparent clean#6508

Open
swapnachoudhary43 wants to merge 4 commits intosugarlabs:masterfrom
swapnachoudhary43:fix/highcontrast-transparent-clean
Open

Fix/highcontrast transparent clean#6508
swapnachoudhary43 wants to merge 4 commits intosugarlabs:masterfrom
swapnachoudhary43:fix/highcontrast-transparent-clean

Conversation

@swapnachoudhary43
Copy link
Copy Markdown
Contributor

What this PR does

Fixes a typo in themes.css where
background-color: transparent was accidentally split
across two lines in the high contrast helpfulSearchDiv style.

This restores the correct transparent background rendering in high contrast mode.

Category

Bug fix

Testing

  • Verified in browser
  • Enabled high contrast mode
  • Confirmed helpfulSearchDiv now renders transparent correctly

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2026

✅ All Jest tests passed! This PR is ready to merge.

@Rudra2637
Copy link
Copy Markdown
Contributor

Add the pr category in the description

Copy link
Copy Markdown
Contributor

@vanshika2720 vanshika2720 left a comment

Choose a reason for hiding this comment

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

@swapnachoudhary43 Add a pr-category

@zealot-zew
Copy link
Copy Markdown
Contributor

Seems to be having unrealated files/changes. can you please remove those.

@@ -0,0 +1,2730 @@
// Copyright (c) 2014-2021 Walter Bender
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

did you just create a complete file?
description is about a typo fix only, right?

@karthik-dev56
Copy link
Copy Markdown
Contributor

Hey @swapnachoudhary43
revert the unrelated files and make sure the themes.css line is set correctly to background-color: transparent ! important Also, add the PR category in your description to pass the test

Copy link
Copy Markdown
Contributor

@vanshika2720 vanshika2720 left a comment

Choose a reason for hiding this comment

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

@swapnachoudhary43 unrelated changes !!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants