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

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

Issue 1281823002: Gallery: change size of crop overlay to cover entire window. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 4 months 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 | « ui/file_manager/gallery/css/gallery.css ('k') | ui/file_manager/gallery/js/image_editor/image_transform.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/file_manager/gallery/js/image_editor/image_editor.js
diff --git a/ui/file_manager/gallery/js/image_editor/image_editor.js b/ui/file_manager/gallery/js/image_editor/image_editor.js
index b2e62adaa19009fde263d2d9edfcd810d2ed31de..96d21362ed968e7849145c7818b796e3ce1fafe4 100644
--- a/ui/file_manager/gallery/js/image_editor/image_editor.js
+++ b/ui/file_manager/gallery/js/image_editor/image_editor.js
@@ -706,34 +706,6 @@ ImageEditor.prototype.onDoubleTap_ = function(x, y) {
};
/**
- * Hide the tools that overlap the given rectangular frame.
- *
- * @param {ImageRect=} opt_frame Hide the tool that overlaps this rect.
- * @param {ImageRect=} opt_transparent But do not hide the tool that is
- * completely inside this rect.
- */
-ImageEditor.prototype.hideOverlappingTools = function(
- opt_frame, opt_transparent) {
- var frame = opt_frame || null;
- var transparent = opt_transparent || null;
-
- var tools = this.rootContainer_.ownerDocument.querySelectorAll('.dimmable');
- var changed = false;
- for (var i = 0; i != tools.length; i++) {
- var tool = tools[i];
- var toolRect = tool.getBoundingClientRect();
- var overlapping =
- (!!frame && frame.intersects(toolRect)) &&
- !(!!transparent && transparent.contains(toolRect));
- if (overlapping && !tool.hasAttribute('dimmed') ||
- !overlapping && tool.hasAttribute('dimmed')) {
- ImageUtil.setAttribute(tool, 'dimmed', overlapping);
- changed = true;
- }
- }
-};
-
-/**
* A helper object for panning the ImageBuffer.
*
* @param {!HTMLElement} rootContainer The top-level container.
« no previous file with comments | « ui/file_manager/gallery/css/gallery.css ('k') | ui/file_manager/gallery/js/image_editor/image_transform.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698