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

Unified Diff: Source/core/inspector/InspectorOverlayPage.html

Issue 1311783003: Devtools[LayoutEditor]: Rework layout-editor workflow (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@resize
Patch Set: Rebase on dgozman changes 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
Index: Source/core/inspector/InspectorOverlayPage.html
diff --git a/Source/core/inspector/InspectorOverlayPage.html b/Source/core/inspector/InspectorOverlayPage.html
index 52fc463cf6e9023fce988a3f5b48beda9333133f..6cd041b877581c00fe54b9f701ddbfcf15ff954a 100644
--- a/Source/core/inspector/InspectorOverlayPage.html
+++ b/Source/core/inspector/InspectorOverlayPage.html
@@ -447,7 +447,12 @@ function reset(resetData)
editor.style.visibility = "hidden";
editor.textContent = "";
document.body.classList.remove("dimmed");
- document.removeEventListener("mousemove", onGlobalMouseMove);
+ document.removeEventListener("mousedown", consumeEvent);
+ document.removeEventListener("mousemove", onLayoutEditorMouseMove);
+ document.removeEventListener("mouseup", consumeEvent);
+ document.removeEventListener("click", onLayoutEditorClick);
+ document.removeEventListener("keydown", onLayoutEditorKeyDown);
+
delete window.hoverableAreas;
delete window.anchorsByType;
window._gridPainted = false;
@@ -810,7 +815,11 @@ function showLayoutEditor(info)
if (selectedAnchorInfo)
showLabels(selectedAnchorInfo.type, selectedAnchorInfo.propertyName);
- document.addEventListener("mousemove", onGlobalMouseMove);
+ document.addEventListener("mousedown", consumeEvent);
+ document.addEventListener("mousemove", onLayoutEditorMouseMove);
+ document.addEventListener("mouseup", consumeEvent);
+ document.addEventListener("click", onLayoutEditorClick);
+ document.addEventListener("keydown", onLayoutEditorKeyDown);
}
function createAnchor(anchorInfo)
@@ -1020,8 +1029,9 @@ function anchorColorForProperty(anchorInfo)
return "rgb(107, 213, 0)";
}
-function onGlobalMouseMove(event)
+function onLayoutEditorMouseMove(event)
{
+ event.preventDefault();
if (!window.hoverableAreas)
return;
var types = ["padding", "margin"];
@@ -1034,6 +1044,26 @@ function onGlobalMouseMove(event)
}
}
+function onLayoutEditorClick(event)
+{
+ event.preventDefault();
+ InspectorOverlayHost.clearSelection(true);
+}
+
+function onLayoutEditorKeyDown(event)
+{
+ // clear selection on ESC
dgozman 2015/08/28 23:44:11 nit: Clear selection on Esc.
sergeyv 2015/09/02 00:40:58 Done.
+ if (event.keyIdentifier === "U+001B") {
+ event.preventDefault();
+ InspectorOverlayHost.clearSelection(false);
+ }
+}
+
+function consumeEvent(event)
+{
+ event.preventDefault();
+}
+
function resetLabelCanvas()
{
delete window.labelCanvasState;

Powered by Google App Engine
This is Rietveld 408576698