New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 900593 link

Starred by 1 user
Status: Assigned
Owner:
pschaffner@chromium.org
Cc:
█ ghendel@chromium.org
pschaffner@chromium.org
justincohen@chromium.org
olivierrobin@chromium.org
linds...@chromium.org
ios-bugs@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 2
Type: Bug



Sign in to add a comment

Splash screen is displayed on launching the app from background

Project Member Reported by rakurati@chromium.org, Oct 31 
App Version: 71.0.3578.28 Beta
iOS Version: 10.3.3, 11.4.1, 12.0.1, 12.1 beta
Device: iPhone and iPad                                                                                                                                                                                                                                                                                        

Steps to reproduce:
1. Launch chrome
2. Open a new incognito tab
3. Keep the incognito tab in foreground
4. Background the app
5. Launch the app (if splash screen is not seen wait for a couple of seconds and launch the app)

Observed results:
Splash screen is displayed

Expected results:
Splash screen shouldn’t be shown on loading app from background

Number of times you were able to reproduce: 5/5
Bug reproducible after clean install: Yes
Bug reproducible after clearing cache and cookies: Yes
Bug reproducible on Chrome Mobile on Android: Not tested
Bug reproducible on Safari/Firefox: Firefox: NA, Safari: NA
Bug reproducible on current stable build (App Version, iOS Version): Yes on M70
Bug reproducible on the current beta channel build (App Version, iOS Version): Yes on M71 Beta

Link to Video:
https://drive.google.com/file/d/15psJ6Yv5gwe8aRQKLLqcdPBVG7j1fO5q/view?usp=sharing

Comment 1 by sczs@chromium.org, Oct 31

Cc: linds...@chromium.org pschaffner@chromium.org olivierrobin@chromium.org justincohen@chromium.org ghendel@chromium.org
Components: -UI>Shell>StartScreen UI>Browser>Incognito
Labels: Needs-Feedback
Owner: mard...@chromium.org
Status: Assigned (was: Untriaged)
Mardini: We're getting feedback that is jarring to go from the white splash to the dark incognito style, could we consider a darker splash screen for the incognito "obscuring feature"

rakurati@: Unless we're miss-understanding something this is intended behavior, is this how M69 (pre-refresh) looked? (The only difference is that we had a different image back then)

Comment 2 by ghendel@google.com, Oct 31 

The old UI also had a white splashscreen, but I agree that it would be nice to change this.

Comment 3 by justincohen@chromium.org, Oct 31 

The old incognito mask was not white, it was the old tab-switcher dark background.

Comment 4 by lindsayw@google.com, Oct 31 

I think going from white animation to black incognito tab also blurs the am-i-in-incognito e2e continuity as well.

Comment 5 by ghendel@google.com, Oct 31 

Attaching a video from the old UI that shows a white interstitial before opening incognito. I thought that was what we were talking about here.
ScreenRecording-10-31-2018 11-02-28.MP4
776 KB View Download

Comment 6 by justincohen@chromium.org, Oct 31 

It looks like you did a cold start.  Instead, try to background the app, then foreground.

Here's a video that shows a cold start (splash screen), then a background/foreground, that shows the old tab-switcher background.
GRIF3906.MP4
5.0 MB View Download

Comment 7 by rakurati@chromium.org, Nov 1

Labels: -Needs-Feedback
Displays a dark background in Old UI. Please find the attached video from 68.0.3440.70 beta

Link to video:
https://drive.google.com/file/d/1MBL1ufMbuZvp5T3wPjR577jAt7ReAq8o/view?usp=sharing

Comment 8 by mard...@chromium.org, Nov 1

Status: Available (was: Assigned)
Yes. I agree this is something that should be fixed.
Thank you. Can it be done for m72?

Comment 9 by mard...@chromium.org, Nov 1

Owner: ----

Comment 10 by justincohen@google.com, Nov 1

Owner: pschaffner@chromium.org
Status: Assigned (was: Available)
pschaffner@ This should be an easy change for M72, provided we have an image.  What do you recommend?

Comment 11 by pschaffner@chromium.org, Nov 5 

I don't understand why we flash that old tab switcher background (seems like a bug). Can't we just remove/fix that so it doesn't occur? If it really must be there, then graphic wise I suggest just replacing the old image view with a 100% black view (no asset required).

Comment 12 by pschaffner@chromium.org, Nov 5 

Sorry about my comments in #11 – got myself confused with comments #5 and 6. Will post back shortly with mocks + specs.
►

Sign in to add a comment