Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(489)

Issue 9049006: [cros] Profile photo caption made multiline. (Closed)

Created:
8 years, 12 months ago by Ivan Korotkov
Modified:
8 years, 12 months ago
Reviewers:
Nikita (slow)
CC:
chromium-reviews, arv (Not doing code reviews), nkostylev+watch_chromium.org
Visibility:
Public.

Description

[cros] Profile photo caption made multiline. BUG=chromium-os:23576 TEST=Manual: see bug description Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=115896

Patch Set 1 #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -3 lines) Patch
M chrome/browser/resources/chromeos/login/oobe.css View 1 chunk +0 lines, -3 lines 3 comments Download

Messages

Total messages: 6 (0 generated)
Ivan Korotkov
PTAL
8 years, 12 months ago (2011-12-28 14:42:23 UTC) #1
Nikita (slow)
lgtm http://codereview.chromium.org/9049006/diff/1/chrome/browser/resources/chromeos/login/oobe.css File chrome/browser/resources/chromeos/login/oobe.css (left): http://codereview.chromium.org/9049006/diff/1/chrome/browser/resources/chromeos/login/oobe.css#oldcode340 chrome/browser/resources/chromeos/login/oobe.css:340: overflow: hidden; How many lines could fit in ...
8 years, 12 months ago (2011-12-28 14:49:42 UTC) #2
Ivan Korotkov
http://codereview.chromium.org/9049006/diff/1/chrome/browser/resources/chromeos/login/oobe.css File chrome/browser/resources/chromeos/login/oobe.css (left): http://codereview.chromium.org/9049006/diff/1/chrome/browser/resources/chromeos/login/oobe.css#oldcode340 chrome/browser/resources/chromeos/login/oobe.css:340: overflow: hidden; On 2011/12/28 14:49:42, Nikita Kostylev wrote: > ...
8 years, 12 months ago (2011-12-28 15:31:56 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/ivankr@chromium.org/9049006/1
8 years, 12 months ago (2011-12-28 15:55:31 UTC) #4
commit-bot: I haz the power
Change committed as 115896
8 years, 12 months ago (2011-12-28 17:18:38 UTC) #5
Nikita (slow)
8 years, 12 months ago (2011-12-28 18:44:43 UTC) #6
http://codereview.chromium.org/9049006/diff/1/chrome/browser/resources/chrome...
File chrome/browser/resources/chromeos/login/oobe.css (left):

http://codereview.chromium.org/9049006/diff/1/chrome/browser/resources/chrome...
chrome/browser/resources/chromeos/login/oobe.css:340: overflow: hidden;
On 2011/12/28 15:31:56, Ivan Korotkov wrote:
> On 2011/12/28 14:49:42, Nikita Kostylev wrote:
> > How many lines could fit in there now? 2 or 3?
> 
> 2. It doesn't look neatly enough IMO, so I'd suggest still shortening the
label.

Please mention that in message description. You could mention that in a way "Max
space for one line message if written in 'w' letters is X". Or a similar way to
provide some guideline how Language specialist could check that.

Powered by Google App Engine
This is Rietveld 408576698