[filebrowser] Yet another animation in gallery ribbon.

[filebrowser] Yet another animation in gallery ribbon.

BUG=chromium-os:23434
TEST=See bug.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=112969

[dgozman, 2011-12-02 15:17:40]

Please, take a look.

[Vladislav Kaznacheev, 2011-12-02 15:45:13]

http://codereview.chromium.org/8770044/diff/1/chrome/browser/resources/file_m...
File chrome/browser/resources/file_manager/js/image_editor/gallery.js (right):

http://codereview.chromium.org/8770044/diff/1/chrome/browser/resources/file_m...
chrome/browser/resources/file_manager/js/image_editor/gallery.js:656: var box =
this.items_[index].getBox(0);
shall we get rid of the second image in the Ribbon.Item?

http://codereview.chromium.org/8770044/diff/1/chrome/browser/resources/file_m...
chrome/browser/resources/file_manager/js/image_editor/gallery.js:657:
box.style['margin-left'] = '0';
why not style.marginLeft?

[dgozman, 2011-12-05 09:16:57]

http://codereview.chromium.org/8770044/diff/1/chrome/browser/resources/file_m...
File chrome/browser/resources/file_manager/js/image_editor/gallery.js (right):

http://codereview.chromium.org/8770044/diff/1/chrome/browser/resources/file_m...
chrome/browser/resources/file_manager/js/image_editor/gallery.js:656: var box =
this.items_[index].getBox(0);
On 2011/12/02 15:45:13, Vladislav Kaznacheev wrote:
> shall we get rid of the second image in the Ribbon.Item?

For sure. Done.

http://codereview.chromium.org/8770044/diff/1/chrome/browser/resources/file_m...
chrome/browser/resources/file_manager/js/image_editor/gallery.js:657:
box.style['margin-left'] = '0';
On 2011/12/02 15:45:13, Vladislav Kaznacheev wrote:
> why not style.marginLeft?

Done.

[Vladislav Kaznacheev, 2011-12-05 09:19:22]

lgtm

[commit-bot: I haz the power, 2011-12-05 09:26:05]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dgozman@chromium.org/8770044/3001

[commit-bot: I haz the power, 2011-12-05 10:37:14]

Change committed as 112969