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

Unified Diff: ui/file_manager/gallery/js/slide_mode.js

Issue 2509223002: Gallery: Correct tab order of modal tool bar in edit mode. (Closed)
Patch Set: Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/file_manager/gallery/js/slide_mode.js
diff --git a/ui/file_manager/gallery/js/slide_mode.js b/ui/file_manager/gallery/js/slide_mode.js
index e74908f0cc13985f58b32b5c6e77d31e206f0ab2..5effd8e40e2ac921a93b9dd751b8c48fd87366bb 100644
--- a/ui/file_manager/gallery/js/slide_mode.js
+++ b/ui/file_manager/gallery/js/slide_mode.js
@@ -409,7 +409,12 @@ function SlideMode(container, content, topToolbar, bottomToolbar, prompt,
* @private
* @const
*/
- this.editBarMode_ = util.createChild(this.container_, 'edit-modal');
+ this.editBarMode_ = document.createElement('div');
+ this.editBarMode_.className = 'edit-modal';
+ // Edito modal should be inserted before the bottom toolbar to make the tab
oka 2016/11/17 07:26:40 typo: Edito
fukino 2016/11/17 08:38:00 Done.
+ // order and visual position consistent.
+ this.container_.insertBefore(
+ this.editBarMode_, document.querySelector('#bottom-toolbar'));
/**
* @type {!HTMLElement}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698