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

Issue 624186 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: Aug 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Uneven spacing between Opt-in IME menu, notification center and uber tray

Project Member Reported by sdantul...@chromium.org, Jun 28 2016

Issue description

Google Chrome	53.0.2773.0 (Official Build) dev (64-bit)
Revision	ced2fcee2c85702055d028f4e3e48c5a75a7e41c-refs/heads/master@{#400610}
Platform	8513.0.0 (Official Build) dev-channel link

What steps will reproduce the problem?
1. Open Settings -> Languages and input section
2. Enable "Show input options in the shelf" setting
3. Observe IME opt-in menu in shelf

What do you see instead?
Uneven spacing between Opt-in menu, notification center and uber tray

Attached screenshot.
 
Screenshot 2016-06-28 at 4.36.25 PM.png
47.7 KB View Download
Owner: azurewei@chromium.org
Status: Assigned (was: Untriaged)
Project Member

Comment 2 by sheriffbot@chromium.org, Jul 1 2016

Labels: -M-53 M-54 MovedFrom-53
Moving this nonessential bug to the next milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 15 2016

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

commit c0b0a4f3a8397cb257d975d947bf035e2b0d9a9e
Author: azurewei <azurewei@chromium.org>
Date: Fri Jul 15 08:56:12 2016

Adds ImeMenuTray element on status area widget for IME menu. It has the following behaviors:

- Lays on the status tray, which has a consistent spacing between other shelf items.
- Unmoveable by users, but automatically moving when status tray show/hide other items, like VK/web notification tray element.
- Show/Hide the icon according to activation of new menu feature.
- Show the short name of the current selected IME.
- Change background color when clicked.

BUG=570761,  624186 
TEST=Verified on local build.

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

[modify] https://crrev.com/c0b0a4f3a8397cb257d975d947bf035e2b0d9a9e/ash/ash.gyp
[modify] https://crrev.com/c0b0a4f3a8397cb257d975d947bf035e2b0d9a9e/ash/ash_chromeos_strings.grdp
[add] https://crrev.com/c0b0a4f3a8397cb257d975d947bf035e2b0d9a9e/ash/common/system/chromeos/ime_menu/ime_menu_tray.cc
[add] https://crrev.com/c0b0a4f3a8397cb257d975d947bf035e2b0d9a9e/ash/common/system/chromeos/ime_menu/ime_menu_tray.h
[add] https://crrev.com/c0b0a4f3a8397cb257d975d947bf035e2b0d9a9e/ash/common/system/chromeos/ime_menu/ime_menu_tray_unittest.cc
[modify] https://crrev.com/c0b0a4f3a8397cb257d975d947bf035e2b0d9a9e/ash/common/system/status_area_widget.cc
[modify] https://crrev.com/c0b0a4f3a8397cb257d975d947bf035e2b0d9a9e/ash/common/system/status_area_widget.h
[modify] https://crrev.com/c0b0a4f3a8397cb257d975d947bf035e2b0d9a9e/ash/common/system/tray/tray_constants.cc
[modify] https://crrev.com/c0b0a4f3a8397cb257d975d947bf035e2b0d9a9e/ash/common/system/tray/tray_constants.h
[modify] https://crrev.com/c0b0a4f3a8397cb257d975d947bf035e2b0d9a9e/ash/test/test_system_tray_delegate.cc
[modify] https://crrev.com/c0b0a4f3a8397cb257d975d947bf035e2b0d9a9e/ash/test/test_system_tray_delegate.h

Status: Fixed (was: Assigned)
Status: Verified (was: Fixed)
Chrome OS 54.0.2840.33/8743.35.0

Sign in to add a comment