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

Domains - TabBar has a transparent background in the detail screen when using Dark Mode #23493

Closed
geriux opened this issue Aug 13, 2024 · 5 comments · Fixed by #23495
Closed
Assignees
Labels
[Type] Bug UI User interface bugs

Comments

@geriux
Copy link
Member

geriux commented Aug 13, 2024

In trunk the TabBar does not have a background when using the app in Dark Mode and accessing the Domain's detail screen.

Expected behavior

The TabBar should have a background in Dark mode

ScreenRecording_08-13-2024.15-18-47_1.MP4

Actual behavior

The TabBar has a transparent background

ScreenRecording_08-13-2024.15-13-39_1.MP4

Steps to reproduce the behavior

  • Open the app
  • Go to "Me" in the TabBar
  • All Domains
  • Open a Domain
Tested on Simulator/iPhone 15 Pro, iOS 17/18.1, Jetpack iOS (trunk)
@geriux geriux added [Type] Bug UI User interface bugs labels Aug 13, 2024
@dangermattic
Copy link
Collaborator

Thanks for reporting! 👍

@geriux
Copy link
Member Author

geriux commented Aug 13, 2024

Hey @kean 👋 I know you've worked recently on some changes related to the navigation. Could this be related?

@kean kean self-assigned this Aug 13, 2024
@kean
Copy link
Contributor

kean commented Aug 13, 2024

Nice catch. Thanks for reporting it. I'll address it shortly.

@kean
Copy link
Contributor

kean commented Aug 13, 2024

Fixed in #23495.

Btw, I'm not sure how you got it to work because this page always redirects me to the standalone Safari because of this code:

            if !self.shouldAllowNavigation(for: url) {
                // Open URL in device browser then go back to Domain Management page.
                self.open(url)
                self.goBack()
            }

It was broken for me since the introduction, so its an existing bug.

@geriux
Copy link
Member Author

geriux commented Aug 14, 2024

Btw, I'm not sure how you got it to work because this page always redirects me to the standalone Safari because of this code:

Interesting, I was able to access all of my domains 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Bug UI User interface bugs
Projects
None yet
3 participants