duckduckgo / Android

DuckDuckGo Android App
https://play.google.com/store/apps/details?id=com.duckduckgo.mobile.android
Apache License 2.0
3.82k stars 901 forks source link

[Bug] Dialog Fragments do not follow Dark theme #1411

Closed 4shutosh closed 3 years ago

4shutosh commented 3 years ago

Describe the bug

DialogFragments (example: from settings activity) do not follow the theming options of the app

How to Reproduce

Open Settings Screen:

Expected behavior

Environment

- DDG App Version: develop branch
- Device: Pixel 4XL
- OS: Android 11
4shutosh commented 3 years ago

I would like to work on this issue. @malmstein Can you please assign this to me?

malmstein commented 3 years ago

Thanks for filing this one @4shutosh!

I appreciate your desire to help, however we'll take this one internally. The main reason being, we need input from the Design team to decide colors and background and an internal review from different stakeholders. It would take too long to go back and forth.

4shutosh commented 3 years ago

Got it @malmstein Thank you for the explanation.

I'll try to look into some issues which won't require an internal process for further continuity :) Also please feel free to assign me any issue/task of your choice, I'll be more than happy to work on it.

cmonfortep commented 3 years ago

This has been already fixed and published in our latest release 5.95.0

4shutosh commented 3 years ago

I've installed 5.95.0 from google playstore and this does not work foe me. Screenshot_20210913-225815_Settings~2 Screenshot_20210913-225828_DuckDuckGo

4shutosh commented 3 years ago

@cmonfortep Can you please corroborate? ( Device: Pixel 5, Android 11 )

cmonfortep commented 3 years ago

Sorry, you are right! Thanks for the screenshot. We fixed the confirmation dialog to clear all data.

I'm reopening the issue.

4shutosh commented 3 years ago

Sure 🙌🏻 Can I please work on this issue 😄 ?

cmonfortep commented 3 years ago

@4shutosh 👍👍