digilib-old: client/digitallibrary/jquery/jquery.digilib.js comparison

comparison client/digitallibrary/jquery/jquery.digilib.js @ 641:8f40f7b6fd20 jquery

some small aboutdiv fixes

author	hertzhaft
date	Wed, 19 Jan 2011 15:53:08 +0100
parents	fcb887110471
children	c108607e1bc0

comparison

equal deleted inserted replaced

-:fcb887110471
+:8f40f7b6fd20
 // create HTML structure
 setupScalerDiv(data);
 setupButtons(data, 'actionsStandard');
 // bird's eye view creation
 if (settings.isBirdDivVisible) {
 setupBirdDiv(data);
 }
 // about window creation - TODO: could be deferred? restrict to only one item?
 setupAboutDiv(data);
 });
 },
 }
 };
 // sets a key to a value (relative values with +/- if relative=true)
 var setNumValue = function(settings, key, value) {
+if (isNumber(value)) return settings[key] = value;
 var sign = value.substring(0,1);
 if (sign === '+' || sign === '-') {
 if (settings[key] == null) {
 // this doesn't make much sense but still...
 settings[key] = 0;
 // display marks
 renderMarks(data);
 // show birds eye view
 //showDiv(settings.isBirdDivVisible);
 //digilib.showArrows(); // show arrow overlays for zoom navigation
-// done -- hide about div
+// done -- hide about div ---
-settings.isAboutDivVisible = showDiv(null, data.$aboutDiv, 0);
+// --- why? This only leads to suprise effects when displayed programmatically
+// settings.isAboutDivVisible = showDiv(null, data.$aboutDiv, 0);
 };
 };
 // place marks on the image
 var renderMarks = function (data) {
 $mark.offset({ left : mpos.x, top : mpos.y});
 }
 }
 };
+// auxiliary function (from Douglas Crockford, A.10)
+var isNumber = function isNumber(value) {
+return typeof value === 'number' && isFinite(value);
+};
 // auxiliary function to crop senseless precision
 var cropFloat = function (x) {
 return parseInt(10000 * x, 10) / 10000;
 };