New issue
Advanced search Search tips

Issue 678815 link

Starred by 2 users
Status: Fixed
Owner:
melandory@chromium.org
Closed: Nov 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Non readable "Saved passwords will appear here" in MultiWindow

Project Member Reported by mar...@mwiacek.com, Jan 5 2017 
Steps to reproduce the problem:
1. Settings -> Save passwords
2. switch app to MultiWindow (checked on Samsung / Android 6, but the same should be on Android N)

What is the expected behavior?
Texts are readable

What went wrong?
Texts are displayed in one screen place

Did this work before? No 

Chrome version: 55  Channel: stable
OS Version: 6
Flash Version:
Screenshot_20170106-003439[1].png
184 KB View Download

Comment 1 by mar...@mwiacek.com, Jan 5 2017 

https://codereview.chromium.org/2613133002/ contains proposed patch.

Screenshot from proposed patch set 1 is attached (on top code with changes, on bottom original version)
Screenshot_20170106-002916[1].png
215 KB View Download

Comment 2 by rsgav...@chromium.org, Jan 6 2017

Cc: aska...@chromium.org
Labels: triage-te

Comment 3 by aska...@chromium.org, Jan 6 2017

Cc: -aska...@chromium.org
Labels: -triage-te
Owner: melandory@chromium.org
Status: Assigned (was: Unconfirmed)
Similar to issue Issue 634082 .
melandory@ - Can you please take a look.
Project Member

Comment 4 by bugdroid1@chromium.org, Jan 10 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/3b94a508d33ecf9687bd6d539dd940bfd89c59e9

commit 3b94a508d33ecf9687bd6d539dd940bfd89c59e9
Author: marcin <marcin@mwiacek.com>
Date: Tue Jan 10 00:09:58 2017

Updating "Settings -> Save passwords" screen for MultiWindow

BUG= 678815 

https://bugs.chromium.org/p/chromium/issues/attachment?aid=265773&inline=1 show screen after changes (on top changed version, on bottom original)

Review-Url: https://codereview.chromium.org/2613133002
Cr-Commit-Position: refs/heads/master@{#442410}

[modify] https://crrev.com/3b94a508d33ecf9687bd6d539dd940bfd89c59e9/chrome/android/java/src/org/chromium/chrome/browser/preferences/password/SavePasswordsPreferences.java

Comment 5 by twelling...@chromium.org, Jan 10 2017 

Issue 620642 has been merged into this issue.

Comment 6 by melandory@chromium.org, Nov 17 2017

Status: Fixed (was: Assigned)
►

Sign in to add a comment