Module: check_mk
Branch: master
Commit: b86be2ee9ab0e4014824ebc303a0d1987b2d5a9a
URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=b86be2ee9ab0e4…
Author: Lars Michelsen <lm(a)mathias-kettner.de>
Date: Tue Jan 4 10:46:49 2011 +0100
Updated bug entries
---
.bugs/19 | 16 +++++++++++++---
1 files changed, 13 insertions(+), 3 deletions(-)
diff --git a/.bugs/19 b/.bugs/19
index b2f380a..25fa115 100644
--- a/.bugs/19
+++ b/.bugs/19
@@ -1,12 +1,22 @@
Title: Kommandobox / Filter soll refresh blockieren
Component: multisite
+State: done
+Class: nastiness
+Date: 2010-12-21 16:21:40
Benefit: 2
-State: open
Cost: 3
-Date: 2010-12-21 16:21:40
-Class: nastiness
+Fun: 0
Wenn man gerade die Kommandobox, die Filter oder die View options auf hat
und der Browser macht den automatischen Refresh, dann geht das wieder zu.
Kann man nicht - wenn man die Box aufklappt - den Refresh für die Zeit
unterdrücken?
+
+2011-01-04 10:46:35: changed state open -> done
+Ist erledigt.
+
+
+View pages are not refreshed when at least one form (Filter, Commands,
+Display Options) is open
+
+Switched page refresh code from HTTP header to javascript timeouts to be more controllable
Module: check_mk
Branch: master
Commit: 76c0c7a60accf1c15ee8d76ec6229b8b081256b9
URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=76c0c7a60accf1…
Author: Lars Michelsen <lm(a)mathias-kettner.de>
Date: Tue Jan 4 11:04:57 2011 +0100
Updated bug entries
---
.bugs/72 | 11 ++++++++---
1 files changed, 8 insertions(+), 3 deletions(-)
diff --git a/.bugs/72 b/.bugs/72
index ee933e1..32428bf 100644
--- a/.bugs/72
+++ b/.bugs/72
@@ -1,11 +1,16 @@
Title: Quicksearch case-insensitive
Component: multisite
+State: done
+Class: feature
+Date: 2010-12-22 11:07:51
Benefit: 1
-State: open
Cost: 1
-Date: 2010-12-22 11:07:51
-Class: feature
+Fun: 0
Evtl. sollte das Case-Insensitive werden. Es spricht doch eigentlich
nix dagegen. Gefordert wird das von Leuten, die ihre Hosts mit Großbuchstaben
schreiben. Ich denke bei den Services ist es jetzt schon ci.
+
+2011-01-04 11:04:51: changed state open -> done
+Original Verhalten wieder hergestellt. Das Snapin war schonmal case insensitive.
+Ist wohl unter die Räder gekommen.
Module: check_mk
Branch: master
Commit: 40a1704c849b1738c4b1ce13c0bcb79ce291651c
URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=40a1704c849b17…
Author: Lars Michelsen <lm(a)mathias-kettner.de>
Date: Tue Jan 4 11:05:09 2011 +0100
Sidebar: The sidebar quicksearch snapin is case insensitive again
---
ChangeLog | 1 +
web/htdocs/js/search.js | 11 +++++++++--
web/plugins/sidebar/search.py | 2 +-
3 files changed, 11 insertions(+), 3 deletions(-)
diff --git a/ChangeLog b/ChangeLog
index 3ffaf27..9a66338 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -39,6 +39,7 @@
* Sidebar: Fixed scrolling problem in IE8 with compat mode (or maybe older IE versions)
which broke the snapin titles and also the tactical overview table
* Sidebar: Fixed bulletlist positioning
+ * Sidebar: The sidebar quicksearch snapin is case insensitive again
* Fixed header displaying on views when the edit button is not shown to the user
* View pages are not refreshed when at least one form (Filter, Commands,
Display Options) is open
diff --git a/web/htdocs/js/search.js b/web/htdocs/js/search.js
index 7c6ba69..4340be0 100644
--- a/web/htdocs/js/search.js
+++ b/web/htdocs/js/search.js
@@ -100,7 +100,7 @@ function mkSearchFindUrl(aSearchObjects, objType, oField) {
for (var i in aSearchObjects) {
var objSite = aSearchObjects[i][0];
var objName = aSearchObjects[i][1];
- if (objName.indexOf(namepart) > -1) {
+ if (mkSearchMatch(objName, namepart)) {
found ++;
if (url != null) { // found second match -> not unique
url = null;
@@ -347,6 +347,12 @@ function mkSearchGetUrl(objType, objName, objSite, numMatches) {
return 'view.py?view_name=servicedesc&service=' + objName + '&site=' + objSite;
}
+// This performs a case insensitive search of a substring in a string
+// Returns true if found and false if not
+function mkSearchMatch(base, search) {
+ return base.toLowerCase().indexOf(search.toLowerCase()) > -1;
+}
+
function mkSearchAddSearchResults(aSearchObjects, objType, val) {
val = mkSearchCleanupString(val, objType);
// Build matching regex
@@ -364,7 +370,8 @@ function mkSearchAddSearchResults(aSearchObjects, objType, val) {
objName = aSearchObjects[i][1];
// if(objName.match(oMatch)) {
- if(objName.indexOf(val) > -1) {
+ // case insensitive search!
+ if(mkSearchMatch(objName, val)) {
var url = mkSearchGetUrl(objType, objName, objSite, 1);
var oResult = {
'id': 'result_' + objName,
diff --git a/web/plugins/sidebar/search.py b/web/plugins/sidebar/search.py
index d721d39..18e84b0 100644
--- a/web/plugins/sidebar/search.py
+++ b/web/plugins/sidebar/search.py
@@ -48,7 +48,7 @@ def render_searchform():
html.write('<div id="mk_side_search" onclick="mkSearchClose();">')
html.write('<input id="mk_side_search_field" type="text" name="search" autocomplete="off" />')
html.write('</div>')
- html.write("<script type='text/javascript' src='search.js'></script>\n")
+ html.write("<script type='text/javascript' src='js/search.js'></script>\n")
html.write("<script type='text/javascript'>\n")
# Store (user) hosts in JS array
Module: check_mk
Branch: master
Commit: ddf0867cbdd1656e7cc00c50414a9de48c81b826
URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=ddf0867cbdd165…
Author: Mathias Kettner <mk(a)mathias-kettner.de>
Date: Tue Dec 28 10:46:11 2010 +0100
Updated bug entries
---
.bugs/35 | 18 +++++++++++++++++-
.bugs/87 | 9 +++++++++
2 files changed, 26 insertions(+), 1 deletions(-)
diff --git a/.bugs/35 b/.bugs/35
index 2526ef6..f6f4478 100644
--- a/.bugs/35
+++ b/.bugs/35
@@ -1,4 +1,4 @@
-Title: View-Editor: Spaltenzahl irgendwie dynamisch machen
+Title: View-Editor: Spalten-Editieren komfortabler machen
Component: multisite
Benefit: 3
State: open
@@ -9,3 +9,19 @@ Class: feature
Das aktuelle Prinzip ist eh doof, weil man keine Spalten in der Mitte
einfügen kann. Das müsste man irgendwie über Javascript hinbekommen.
Auch dass man nicht auf 10 begrenzt ist.
+
+Was fehlt ist:
+- einfügen von Spalten
+- Umsortieren von Spalten (noch oben/unten verschieben)
+
+Problem dabei: Die HTML-Variablen sind durchnummeriert. Die
+Nummern bleiben erhalten, selbst wenn man die Elemente im
+DOM-Baum verschiebt.
+
+Wenn man die Nummern im DOM-Baum nicht anpassen kann, dann
+bietet sich vielleicht als Lösung, dass alle Elemente die
+*gleichen* HTML-Variablen verwenden. Diese kommen ja bei
+Apache trotzdem an. Und dass man dann nicht html.var()
+verwendet, sondern eine Hilfsfunktion, die mehrfach genannte
+Variablen konserviert und z.B. als Array zur Verfügung stellt.
+
diff --git a/.bugs/87 b/.bugs/87
new file mode 100644
index 0000000..0f0561e
--- /dev/null
+++ b/.bugs/87
@@ -0,0 +1,9 @@
+Title: TEST
+Component: core
+Benefit: 1
+State: open
+Cost: 1
+Date: 2010-12-28 10:41:33
+Class: todo
+
+TEST
Module: check_mk
Branch: master
Commit: d2d843d723b2ee58f69431cb568864b7271f5c2b
URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=d2d843d723b2ee…
Author: Mathias Kettner <mk(a)mathias-kettner.de>
Date: Mon Dec 27 13:02:15 2010 +0100
Updated some bug entries
---
.bugs/54 | 2 +-
.bugs/57 | 11 ++++++++---
.bugs/86 | 15 +++++++++++++++
3 files changed, 24 insertions(+), 4 deletions(-)
diff --git a/.bugs/54 b/.bugs/54
index a953314..072878e 100644
--- a/.bugs/54
+++ b/.bugs/54
@@ -1,4 +1,4 @@
-Title: \Precompile und cmk -N parallelisieren
+Title: Precompile und cmk -N parallelisieren
Component: core
Benefit: 1
State: open
diff --git a/.bugs/57 b/.bugs/57
index 59b8c94..fb02025 100644
--- a/.bugs/57
+++ b/.bugs/57
@@ -10,9 +10,14 @@ Hover: Wenn man über einen Painter geht, könnte man eine
Hover aufmachen, der einen weiteren Painter anzeigt. Das macht man
über eine Auswahlbox analog zu der "Link to".
-Kritisch ist hier wahrscheinlich hauptsächlich die Frage, wie man
-das im View-Editor noch sinnvoll unterbringt. Die ganze Spalten-geschicht
-muss man dann wohl endgültig neu aufbauen.
+Kritisch ist hier wahrscheinlich hauptsächlich die Frage, wie man das im
+View-Editor noch sinnvoll unterbringt. Die ganze Spalten-geschicht muss man
+dann wohl endgültig neu aufbauen. Man kann an dieser Stelle noch überlegen,
+dass man neben dem Hover-Painter auch Hover-Views einbaut (siehe #0086).
In so einem Hover-Ding könnte man aber viele Interessante Dinge
unterbringen, z.B. host notes, PNP popups und vieles mehr.
+
+Stand jetzt: Halb programmiert. Man kann Tooltips konfigurieren.
+Der Editor schaut noch doof aus. Die Builtin-Views müssen noch
+mit Tooltips ausgestattet werden, wo sinnvoll.
diff --git a/.bugs/86 b/.bugs/86
new file mode 100644
index 0000000..4958adc
--- /dev/null
+++ b/.bugs/86
@@ -0,0 +1,15 @@
+Title: Hover-Preview von View
+Component: multisite
+Benefit: 3
+State: open
+Cost: 3
+Date: 2010-12-26 14:45:15
+Class: feature
+
+Idee: Man könnte zu einem Painter ein Hover-Preview definieren: einfach
+eine andere View (analog zu Kontextlink). Anstelle eines Links wird aber
+ein Hover eingebaut, so dass man den Inhalt der View (nur die Tabelle)
+per Popup bekommt. Dazu mache ich eine spezielle URL, über die man eine
+View als Popup bekommt. Andererseits geht das ja auch schon jetzt mit
+den display_options. Zusätzlich bräuchte man noch eine Option, damit
+die Schriftart kleiner wird oder sowas.