From 00afb047e301aaa17609455bd93c28b332f8d2cc Mon Sep 17 00:00:00 2001 From: "Alex A. Naanou" Date: Tue, 7 Jan 2014 23:10:50 +0400 Subject: [PATCH] added panel visual indicator on open (not sure if the trigger belongs where it currently is)... Signed-off-by: Alex A. Naanou --- ui/TODO.otl | 8 ++++---- ui/css/editor.css | 9 +++++++++ ui/experiments/panels.html | 2 +- ui/lib/panels.js | 15 ++++++++++++++- 4 files changed, 28 insertions(+), 6 deletions(-) diff --git a/ui/TODO.otl b/ui/TODO.otl index 4b8cd170..c8f78bba 100755 --- a/ui/TODO.otl +++ b/ui/TODO.otl @@ -127,7 +127,7 @@ Roadmap [X] bookmarked (list) operations [X] mark/unmark tagged [X] crop - [_] 33% Panels + [_] 34% Panels [X] 100% panel constructors [X] panel [X] side-panel @@ -135,12 +135,12 @@ Roadmap [_] 0% panel state [_] track panel state [_] use panel state for defaults when opening panels - [_] 66% actions - [_] 66% openPanel + [_] 75% actions + [X] 100% openPanel [X] if panel is not loaded load it [X] if a panel is not open, open it and all parents | XXX need to open auto-hidden side panels... - [_] focus panel (???) + [X] blink panel [_] openPanels [X] closePanel | collapse a panel... diff --git a/ui/css/editor.css b/ui/css/editor.css index a0cf2177..d87078b3 100755 --- a/ui/css/editor.css +++ b/ui/css/editor.css @@ -77,6 +77,9 @@ border: solid 1px silver; box-shadow: none; } +.sub-panel.blink { + box-shadow: 0px 0px 10px 0px rgba(255,0,0,1) +} .sub-panel summary { background: #ddd; /* @@ -261,6 +264,9 @@ .dark .sub-panel { border: solid 1px #333; } +.dark .sub-panel.blink { + box-shadow: 0px 0px 10px 0px rgba(255,255,0,1) +} .dark .sub-panel summary { background: #333; color: silver; @@ -339,6 +345,9 @@ .gray .sub-panel { border: solid 1px #454545; } +.gray .sub-panel.blink { + box-shadow: 0px 0px 10px 0px rgba(255,255,0,1) +} .gray .sub-panel summary { background: #444; color: silver; diff --git a/ui/experiments/panels.html b/ui/experiments/panels.html index 916b2268..8cd72efe 100755 --- a/ui/experiments/panels.html +++ b/ui/experiments/panels.html @@ -97,7 +97,7 @@ $(function(){ -

click here to toggle panel autohide and here to toggle theme...

+

click to show the editor panel and here to toggle theme...

diff --git a/ui/lib/panels.js b/ui/lib/panels.js index 9197c138..86a699ca 100755 --- a/ui/lib/panels.js +++ b/ui/lib/panels.js @@ -141,6 +141,16 @@ function getPanel(title){ } +function blinkPanel(panel){ + panel + .addClass('blink') + setTimeout(function(){ + panel.removeClass('blink') + }, 170) + return panel +} + + /********************************************************************** * Constructors... @@ -470,7 +480,10 @@ function openPanel(panel){ panel.trigger('panelOpening', panel) } - return panel + //return panel + // XXX should we be blinking here??? + // ...is this the right context for this??? + return blinkPanel(panel) }