Skip to content

fix(duckduckgo.com): misc #1713

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

undisassemble
Copy link

🔧 What does this fix? 🔧

Fixes unthemed elements on duckduckgo.com, start.duckduckgo.com, duck.ai, and html.duckduckgo.com.
Closes #1514

🗒 Checklist 🗒

@undisassemble undisassemble requested a review from GenShibe as a code owner April 19, 2025 05:24
@github-actions github-actions bot added the duckduckgo DuckDuckGo label Apr 19, 2025
@@ -1237,7 +1236,7 @@
--theme-button-primary-bg: @blue !important;
--theme-searchbox-caret-bg: @accent !important;

.home_root___gvmE {
body div div {
Copy link
Member

Choose a reason for hiding this comment

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

This is better, but are there no other identifying attributes we can use for this second div here?

Copy link
Member

@uncenter uncenter left a comment

Choose a reason for hiding this comment

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

Looks like we could probably put all variables under main, #__next div or something to avoid duplicating them.

Comment on lines +1312 to +1313
color-scheme: if(@flavor = latte, light, dark);

Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
color-scheme: if(@flavor = latte, light, dark);

The previous @-moz-document already covers this and the rest of the default styles, can be removed.

@@ -203,6 +208,13 @@
border-color: @surface0;
}

main {
--theme-howitworks-downloads-bg: @surface0 !important;
--theme-bg-cta-cards: @surface0 !important;
Copy link
Member

Choose a reason for hiding this comment

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

Already set on line 116?

main {
--theme-howitworks-downloads-bg: @surface0 !important;
--theme-bg-cta-cards: @surface0 !important;
--theme-bg-home: @base !important;
Copy link
Member

Choose a reason for hiding this comment

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

Already set under #__next div on line 1240?

--theme-howitworks-downloads-bg: @surface0 !important;
--theme-bg-cta-cards: @surface0 !important;
--theme-bg-home: @base !important;
--theme-bg-home-searchbox: @surface0 !important;
Copy link
Member

Choose a reason for hiding this comment

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

Already set under #__next div on line 1243?

@uncenter uncenter changed the title fix(duckduckgo.com): Fix unthemed elements fix(duckduckgo.com): misc May 1, 2025
@uncenter uncenter added the waiting on author Note for staff that a re-review is not yet required label May 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duckduckgo DuckDuckGo waiting on author Note for staff that a re-review is not yet required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Duckduckgo AI chat: unreadable parts where the background has light color
2 participants