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

Dark theme readability feedback and API bugs issue #5375

Closed
3 tasks done
ainar-g opened this issue Jan 19, 2023 · 24 comments
Closed
3 tasks done

Dark theme readability feedback and API bugs issue #5375

ainar-g opened this issue Jan 19, 2023 · 24 comments

Comments

@ainar-g
Copy link
Contributor

ainar-g commented Jan 19, 2023

Prerequisites

  • I have checked the Wiki and Discussions and found no answer

  • I have searched other issues and found no duplicates

  • I want to request a feature or enhancement and not ask a question

Description

In this issue we would like to gather feedback regarding the dark theme (#613).

Please read the following instructions before posting!

  1. do not post feedback based on personal taste (i.e. “I don't like this colour”, etc.);
  2. do post feedback regarding contrast and readability issues (i.e. “the text in this dialog becomes barely readable”) or API breaking;
  3. do look through the previous comments to make sure someone hadn't made one about the same issue before;
  4. add a screenshot or a video of the issue;
  5. add your browser and OS information;
@Birbber
Copy link

Birbber commented Jan 19, 2023

@ainar-g
Copy link
Contributor Author

ainar-g commented Jan 19, 2023

It seems like we forgot to include the updates locales for the theme selector into the release. This should be fixed as well.

The beta version v0.108.0-b.25 does include them, so those who require it can temporarily switch to that version.

@WildByDesign
Copy link

WildByDesign commented Jan 19, 2023

The only issue that I noticed so far is that the initial Login page does not have dark mode yet even after dark has been manually selected.

Great work, AdGuard Team. Everything looks quite good so far.

@Alex-302
Copy link
Member

The highlighting of blocked requests is too pale.

Details

image

@ameshkov
Copy link
Member

ameshkov commented Jan 19, 2023

Flashing when navigating to the dashboard:
https://uploads.adguard.org/up04_screencast_gj29c.mp4

@CodeSapiens
Copy link

CodeSapiens commented Jan 19, 2023

I appreciate the work and time involved in making this change, but I believe it should be a configurable option.
Perhaps it should be under Settings -> General Settings?

@ainar-g
Copy link
Contributor Author

ainar-g commented Jan 19, 2023

@CodeSapiens, the theme selector is at the bottom of the page, near the language selector.

@CodeSapiens
Copy link

Screenshot 2023-01-19 083203
It does not show up for me.

@ainar-g ainar-g pinned this issue Jan 19, 2023
@ainar-g
Copy link
Contributor Author

ainar-g commented Jan 19, 2023

It should be there, unless you're logged out:

Screenshot at 2023-01-19 17-40-10

Are you sure that that isn't a browser cache issue? How did you install AdGuard Home? If you're using something like the Home Assistant plugin, they usually need some time to update their APIs as well. Also, logging out and logging in might help as well.

@CodeSapiens
Copy link

I have closed the browser, cleared cookies, restarted the router, and still no joy. I have two AGH installations, and only the one installed on a GL.iNet router (OpenWRT) prompted me to upgrade to the new version. The other installation is on a Raspberry Pi, also OpenWRT, and that one is still running v0.107.21 and does not show an icon to check for updates at the bottom of the page. The updated OpenWRT package list still shows 0.107.21-1 as the latest version, which as you suggested, is most likely due to their APIs not being updated yet.

Question: Is there something I can change on the Raspberry Pi installation to force AGH to check for updates automatically?

Thanks for your time Ainar-g.

@ainar-g
Copy link
Contributor Author

ainar-g commented Jan 19, 2023

GL.iNer routers may have their own authorization scheme, so that might be it. We'll investigate that, thanks. In the meantime you can stop your AGH and edit the configuration file to say:

#
theme: light

Question: Is there something I can change on the Raspberry Pi installation to force AGH to check for updates automatically?

Such questions are better discussed in the GitHub Discussions. And that may have something to do with packaging.

@CodeSapiens
Copy link

Thanks again for your time.

@Alex-302
Copy link
Member

Dark theme applies too late after refreshing a page

Details

image

@Alex-302
Copy link
Member

Font in navbar is too dark

Details

image

@Birbber
Copy link

Birbber commented Jan 20, 2023

When checking filters updates 'Loading' is too dark: https://uploads.adguard.org/annrgy_pemty.mp4

@Hanif-Faez
Copy link

The queries dropdown menu is barely readable

queries_filter

@EugeneOne1
Copy link
Member

The coloring of log entries for rewritten requests makes the IP of a client unreadable.
image

@Skyxim
Copy link

Skyxim commented Jan 30, 2023

image
I had a problem viewing the details log with the font reversed and the background still white. It works the same way on my phone

@unknown4849
Copy link

in firefox seem imposible to read this.
image

but in Chromium base (brave , edge) can be read but text gray and white background
image

adguard pushed a commit that referenced this issue Feb 1, 2023
Updates #5375

Squashed commit of the following:

commit 40666b0
Author: Arseny Lisin <a.lisin@adguard.com>
Date:   Tue Jan 31 18:34:06 2023 +0200

    Review fix

commit 44f3d60
Author: Arseny Lisin <a.lisin@adguard.com>
Date:   Tue Jan 31 17:02:38 2023 +0200

    Clear

commit 44274ba
Author: Arseny Lisin <a.lisin@adguard.com>
Date:   Tue Jan 31 13:25:01 2023 +0200

    Clear

commit 8b48c52
Author: Arseny Lisin <a.lisin@adguard.com>
Date:   Tue Jan 31 12:14:37 2023 +0200

    Review fix

commit 3b8cd94
Author: Arseny Lisin <a.lisin@adguard.com>
Date:   Mon Jan 30 16:13:15 2023 +0200

    Fix query log popup bg

commit 14d4c87
Author: Arseny Lisin <a.lisin@adguard.com>
Date:   Mon Jan 30 15:03:06 2023 +0200

    revert icons

commit 98b0427
Author: Arseny Lisin <a.lisin@adguard.com>
Date:   Mon Jan 30 14:45:35 2023 +0200

    Fix dark theme bugs
@ainar-g ainar-g unpinned this issue Feb 1, 2023
@ainar-g
Copy link
Contributor Author

ainar-g commented Feb 1, 2023

Most of the issues should now be fixed on the Edge channel, and the fixes will be shipped in the upcoming release. There are some elements which aren't as easily styled, and we'll try to figure them out later. If anyone finds other issues, please report them separately. Thanks everyone for the feedback!

@ngosang
Copy link

ngosang commented Feb 2, 2023

Great work but there are still some issues.
image

@ameshkov
Copy link
Member

ameshkov commented Feb 3, 2023

@ngosang could you please explain what was there in the light theme?

@ngosang
Copy link

ngosang commented Feb 3, 2023

image

@ameshkov
Copy link
Member

ameshkov commented Feb 3, 2023

@ngosang thank you! Opened a new issue: #5442

heyxkhoa pushed a commit to heyxkhoa/AdGuardHome that referenced this issue Mar 20, 2023
Updates AdguardTeam#5375

Squashed commit of the following:

commit 40666b0
Author: Arseny Lisin <a.lisin@adguard.com>
Date:   Tue Jan 31 18:34:06 2023 +0200

    Review fix

commit 44f3d60
Author: Arseny Lisin <a.lisin@adguard.com>
Date:   Tue Jan 31 17:02:38 2023 +0200

    Clear

commit 44274ba
Author: Arseny Lisin <a.lisin@adguard.com>
Date:   Tue Jan 31 13:25:01 2023 +0200

    Clear

commit 8b48c52
Author: Arseny Lisin <a.lisin@adguard.com>
Date:   Tue Jan 31 12:14:37 2023 +0200

    Review fix

commit 3b8cd94
Author: Arseny Lisin <a.lisin@adguard.com>
Date:   Mon Jan 30 16:13:15 2023 +0200

    Fix query log popup bg

commit 14d4c87
Author: Arseny Lisin <a.lisin@adguard.com>
Date:   Mon Jan 30 15:03:06 2023 +0200

    revert icons

commit 98b0427
Author: Arseny Lisin <a.lisin@adguard.com>
Date:   Mon Jan 30 14:45:35 2023 +0200

    Fix dark theme bugs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests