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

Issue 690563 link

Starred by 5 users

Issue metadata

Status: Verified
Owner:
Closed: Feb 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Language code font is small on system tray

Project Member Reported by songsuk@chromium.org, Feb 9 2017

Issue description

Chrome Version       : 	58.0.3007.0
Platform             : Chromeos 9267.0.0 - Peppy


What steps will reproduce the problem?
(1)   add CJK and UK/French IMEs from chrome://settings
(2)   check the language code font size from System tray


What is the expected result?  
Need to increase font size, m57.png.

What happens instead?
The font size is small. m58.png.  Hardly read the "INTL" characters.

Please provide any additional information below. Attach a screenshot if
possible.

 
m58.png
1.1 MB View Download
m57.png
981 KB View Download
Owner: azurewei@chromium.org
Status: Assigned (was: Untriaged)
Issue 690719 has been merged into this issue.
Status: Started (was: Assigned)
Cc: kavvaru@chromium.org durga.behera@chromium.org brajkumar@chromium.org tbuck...@chromium.org ajha@chromium.org
 Issue 690850  has been merged into this issue.
Screenshots of opt-in IME menu and system IME menu with cl:
https://codereview.chromium.org/2685273002/


opt-in-ime-menu.png
22.9 KB View Download
system-menu.png
25.1 KB View Download
Looking good thanks for the fix :)
Project Member

Comment 7 by bugdroid1@chromium.org, Feb 14 2017

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

commit 436c4bebb1f31ef7246eb7747eccd9a812f09cf7
Author: azurewei <azurewei@chromium.org>
Date: Tue Feb 14 03:57:42 2017

For IMEs whose short name are more than 2 characters (INTL, EXTD, etc.),
we shrink the font size until fits within the bounds of |kMenuIconSize|.
But when performing the shrinking operation, we should use the text size
without insects for comparing.

Verified with both opt-in IME menu and system menu.

BUG= 690563 
TEST=Verified on local build.

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

[modify] https://crrev.com/436c4bebb1f31ef7246eb7747eccd9a812f09cf7/ash/common/system/chromeos/ime_menu/ime_list_view.cc

Status: Fixed (was: Started)
Status: Verified (was: Fixed)

Sign in to add a comment