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

Issue 766898 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Sep 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 3
Type: Bug



Sign in to add a comment

[UX Review] Need to reduce the host list font size

Project Member Reported by yuweih@chromium.org, Sep 20 2017

Issue description

It should be smaller than the title font to give a better sense of hierarchy and fit more characters on the screen for long names.
 

Comment 1 by yuweih@chromium.org, Sep 20 2017

Cc: dah...@chromium.org
WDYT?
文件_000.png
207 KB View Download

Comment 2 by yuweih@chromium.org, Sep 20 2017

We could also make the host title lighter.

Note that we got excessive top padding on the host list on iOS 11. This bug is tracked at  BUG 766892 .
文件_000.png
206 KB View Download

Comment 3 by dah...@chromium.org, Sep 20 2017

The darker version LGTM.
Project Member

Comment 4 by bugdroid1@chromium.org, Sep 22 2017

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

commit 4f6144c6de4988f43915602b257cd9b4c4d36273
Author: Yuwei Huang <yuweih@chromium.org>
Date: Fri Sep 22 19:19:56 2017

[CRD iOS] Reduce host list font size

Screenshot:
https://drive.google.com/a/google.com/file/d/0B8Lt8bnh-turSXhWVl93SWZ5U0U/view?usp=sharing

Bug:  766898 
Change-Id: I48aabd7a97642900df379786d54b98c6b1c9e93f
Reviewed-on: https://chromium-review.googlesource.com/674091
Reviewed-by: Scott Nichols <nicholss@chromium.org>
Commit-Queue: Yuwei Huang <yuweih@chromium.org>
Cr-Commit-Position: refs/heads/master@{#503819}
[modify] https://crrev.com/4f6144c6de4988f43915602b257cd9b4c4d36273/remoting/ios/app/host_collection_view_cell.mm

Comment 5 by yuweih@chromium.org, Sep 22 2017

Status: Fixed (was: Assigned)
Status: Verified (was: Fixed)
Verified Fixed in 34.0.3261.0
Is it 64.0.3261.0?
ah sorry, fat finger... yes, 64.0.3261.0

Sign in to add a comment