Issue 129117 in chromium: IME menu does not fit into the tray

3 views
Skip to first unread message

chro...@googlecode.com

unread,
May 22, 2012, 12:26:26 AM5/22/12
to chromi...@chromium.org
Status: Assigned
Owner: mu...@chromium.org
CC: sad...@chromium.org, n...@chromium.org, zo...@chromium.org
Labels: Type-Bug Pri-2 Area-UI OS-Chrome Mstone-21 Feature-Ash-Tray
Feature-TextInput

New issue 129117 by yus...@chromium.org: IME menu does not fit into the
tray
http://code.google.com/p/chromium/issues/detail?id=129117

Google Chrome 21.0.1145.0 (Official Build 138071) unknown (tot M21)
Platform 2331.0.2012_05_21_1809 (Developer Build - yusukes) developer-build
x86-alex
WebKit 537.1 (trunk@116448)

What steps will reproduce the problem?

1. enable the Japanese IME, switch to it
2. click Omnibox
3. open the system tray, then click "Google Japanese Input".

What is the expected result?

The IME menu which has strings like "Direct input", "Hiragana", .. fits
into the panel.

What happens instead?

It does not. See the screen shot.

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

Please also check M20.


Attachments:
Screenshot 2012-05-21 9:20:04 PM.png 343 KB

chro...@googlecode.com

unread,
May 22, 2012, 1:45:43 PM5/22/12
to chromi...@chromium.org

Comment #1 on issue 129117 by mu...@chromium.org: IME menu does not fit
Reproducing on my environment.
Note that it doesn't reproduce on 21.0.1136.0 (Official Build 136771)
canary, so It's unlikely to happen on M20.

chro...@googlecode.com

unread,
May 22, 2012, 1:48:43 PM5/22/12
to chromi...@chromium.org

Comment #2 on issue 129117 by sad...@chromium.org: IME menu does not fit
I think you can just add the properties in the same container as the IME
list (i.e. in |scroll_content()| instead of creating a new |properties|
container in AppendIMEProperties

chro...@googlecode.com

unread,
May 23, 2012, 8:01:32 PM5/23/12
to chromi...@chromium.org
Updates:
Status: Fixed

Comment #5 on issue 129117 by mu...@chromium.org: IME menu does not fit
(No comment was entered for this change.)

Reply all
Reply to author
Forward
0 new messages