- **BC break:** při použití metody setOptions s parametrem `options…

…` je nově nutné poslat jako druhý parametr `true` pro rozlišení mezi skutečným options a contextem - do callbacku `onLoad` se předává element, který otevřel pdbox, stejně jako do ostatních callbacků
peckadesign · Mar 2, 2018 · ebe20c9 · ebe20c9
1 parent 9451e20
commit ebe20c9
Show file tree

Hide file tree

Showing 6 changed files with 42 additions and 26 deletions.
diff --git a/README.md b/README.md
@@ -17,11 +17,13 @@ $ cat bower.json
 
 ## Changelog
 
-### v1.1.2 draft
+### v1.2.0
+- **BC break:** při použití metody `setOptions` s parametrem `options` je nově nutné poslat jako druhý parametr `true` pro rozlišení mezi skutečným options a contextem
 - oprava autora v `package.json`
 - při použití metody `openHtml` opraveno probliknutí spinneru na pozadí
 - při použití metody `openUrl` je možno předat element, ze kterého se přeberou options (napr. odkaz, na který se kliklo)
 - oprava [#9](https://github.com/peckadesign/jquery.pdbox/issues/9)
+- do callbacku `onLoad` se předává element, který otevřel pdbox, stejně jako do ostatních callbacků
 
 ### v1.1.1
 - oprava [#8](https://github.com/peckadesign/jquery.pdbox/issues/8)

diff --git a/bower.json b/bower.json
@@ -1,6 +1,6 @@
 {
 	"name": "jquery.pdbox",
-	"version": "1.1.2-draft",
+	"version": "1.2.0",
 	"homepage": "https://github.com/peckadesign/jquery.pdbox",
 	"authors": [
 		"PeckaDesign, s.r.o <support@peckadesign.cz>"

diff --git a/dist/js/jquery.pdbox.js b/dist/js/jquery.pdbox.js
@@ -6,7 +6,7 @@
  * @copyright Copyright (c) 2014-2018 PeckaDesign, s.r.o
  * @license MIT
  *
- * @version 1.1.2-draft
+ * @version 1.2.0
  */
 $.pdBox = (function () {
 
@@ -117,7 +117,7 @@ $.pdBox = (function () {
 			"	<div class='pdbox__content'>" +
 			"		<h2 class='pdbox__title'></h2>" +
 			"		<div class='pdbox__desc'>" +
-						(box.isAjax ? "<div id='snippet--pdbox' class='pdbox__snippet'></div>" : "") +
+			(box.isAjax ? "<div id='snippet--pdbox' class='pdbox__snippet'></div>" : "") +
 			"		</div>" +
 			"		<p class='pdbox__pager'>" +
 			"			<a href='#' class='pdbox__page pdbox__page--prev' rel=''><span>" + box.langs[box.options.lang]["prev"] + "</span></a>" +
@@ -127,7 +127,7 @@ $.pdBox = (function () {
 			"		</p>" +
 			"		<p class='pdbox__image'></p>" +
 			"		<div class='pdbox__pager--thumbnails'></div>" +
-					box.spinnerHtml +
+			box.spinnerHtml +
 			"		<a href='#' class='pdbox__close' title='" + box.langs[box.options.lang]["close"] + "'> " + box.langs[box.options.lang]["close"] + "</a>" +
 			"	</div>" +
 			"</div>";
@@ -224,9 +224,10 @@ $.pdBox = (function () {
 	 * Nastaví vlastnosti pdboxu v pořadí defaults, nastavení z inicializace, nastavení z data atributů elementu,
 	 * který tb otevřel. Pokud je předán atribut options, použije se nsatavení z tohoto atributu.
 	 *
-	 * @param options
+	 * @param options		předané options
+	 * @param isOptions		flag, zda jde opravdu o options a ne kontext předaný z onLoad callbacku
 	 */
-	PdBox.prototype.setOptions = function (options) {
+	PdBox.prototype.setOptions = function (options, isOptions) {
 		var e;
 		var optName;
 		var dataName;
@@ -238,7 +239,7 @@ $.pdBox = (function () {
 
 			// slouží k ad-hoc handlerům, proto můžeme smazat
 			for (e in events) {
-				 optName = getEventOptName(events[e]);
+				optName = getEventOptName(events[e]);
 
 				this.options[optName] = null;
 			}
@@ -261,12 +262,13 @@ $.pdBox = (function () {
 			$.extend(this.options, elOptions);
 		}
 
-		if (options) {
+		// pokud byly předány options, ověříme, že jde skutečně o options a ne o context předaný v rámci onLoad callbacku
+		if (options && isOptions) {
 			$.extend(this.options, options);
 		}
 
 		this.rootElem
-			// odstraníme všechny class krom "pdbox" a případné "pdbox--loading"
+		// odstraníme všechny class krom "pdbox" a případné "pdbox--loading"
 			.removeClass(function(i, className) {
 				var list = className.split(' ');
 				return list.filter(function(val){
@@ -360,7 +362,7 @@ $.pdBox = (function () {
 			this.window.title.hide();
 		}
 
-		this.dispatchEvent('load', {content: html});
+		this.dispatchEvent('load', {element: $el, content: html});
 	};
 
 	PdBox.prototype.openUrl = function (href, $el) {
@@ -371,7 +373,7 @@ $.pdBox = (function () {
 			success: $.proxy(function (content) {
 				this.window.descWrap.show();
 				this.window.desc.html(content);
-				this.dispatchEvent('load', {content: content});
+				this.dispatchEvent('load', {element: $el, content: content});
 			}, this)
 		});
 	};
@@ -562,7 +564,7 @@ $.pdBox = (function () {
 			}
 
 			if (e.type === 'load') {
-				box.dispatchEvent('load', {content: preloader});
+				box.dispatchEvent('load', {element: $el, content: preloader});
 			}
 		});
 

diff --git a/dist/js/jquery.pdbox.min.js b/dist/js/jquery.pdbox.min.js
diff --git a/package.json b/package.json
@@ -2,7 +2,7 @@
 	"name": "jquery.pdbox",
 	"title": "jQuery pdBox",
 	"description": "pdBox is thickbox-like jQeruy plugin developed in PeckaDesign",
-	"version": "1.1.2-draft",
+	"version": "1.2.0",
 	"author": {
 		"name":"PeckaDesign, s.r.o",
 		"email": "support@peckadesign.cz"

diff --git a/src/js/jquery.pdbox.js b/src/js/jquery.pdbox.js
@@ -1,3 +1,13 @@
+/**
+ * jQuery pdBox - pdBox is thickbox-like jQeruy plugin developed in PeckaDesign
+ * https://github.com/peckadesign/jquery.pdbox
+ *
+ * @author PeckaDesign, s.r.o <support@peckadesign.cz>
+ * @copyright Copyright (c) 2014-2018 PeckaDesign, s.r.o
+ * @license MIT
+ *
+ * @version 1.1.2-draft
+ */
 $.pdBox = (function () {
 
 	/*************** configuration ***************/
@@ -107,7 +117,7 @@ $.pdBox = (function () {
 			"	<div class='pdbox__content'>" +
 			"		<h2 class='pdbox__title'></h2>" +
 			"		<div class='pdbox__desc'>" +
-						(box.isAjax ? "<div id='snippet--pdbox' class='pdbox__snippet'></div>" : "") +
+			(box.isAjax ? "<div id='snippet--pdbox' class='pdbox__snippet'></div>" : "") +
 			"		</div>" +
 			"		<p class='pdbox__pager'>" +
 			"			<a href='#' class='pdbox__page pdbox__page--prev' rel=''><span>" + box.langs[box.options.lang]["prev"] + "</span></a>" +
@@ -117,7 +127,7 @@ $.pdBox = (function () {
 			"		</p>" +
 			"		<p class='pdbox__image'></p>" +
 			"		<div class='pdbox__pager--thumbnails'></div>" +
-					box.spinnerHtml +
+			box.spinnerHtml +
 			"		<a href='#' class='pdbox__close' title='" + box.langs[box.options.lang]["close"] + "'> " + box.langs[box.options.lang]["close"] + "</a>" +
 			"	</div>" +
 			"</div>";
@@ -214,9 +224,10 @@ $.pdBox = (function () {
 	 * Nastaví vlastnosti pdboxu v pořadí defaults, nastavení z inicializace, nastavení z data atributů elementu,
 	 * který tb otevřel. Pokud je předán atribut options, použije se nsatavení z tohoto atributu.
 	 *
-	 * @param options
+	 * @param options		předané options
+	 * @param isOptions		flag, zda jde opravdu o options a ne kontext předaný z onLoad callbacku
 	 */
-	PdBox.prototype.setOptions = function (options) {
+	PdBox.prototype.setOptions = function (options, isOptions) {
 		var e;
 		var optName;
 		var dataName;
@@ -228,7 +239,7 @@ $.pdBox = (function () {
 
 			// slouží k ad-hoc handlerům, proto můžeme smazat
 			for (e in events) {
-				 optName = getEventOptName(events[e]);
+				optName = getEventOptName(events[e]);
 
 				this.options[optName] = null;
 			}
@@ -251,12 +262,13 @@ $.pdBox = (function () {
 			$.extend(this.options, elOptions);
 		}
 
-		if (options) {
+		// pokud byly předány options, ověříme, že jde skutečně o options a ne o context předaný v rámci onLoad callbacku
+		if (options && isOptions) {
 			$.extend(this.options, options);
 		}
 
 		this.rootElem
-			// odstraníme všechny class krom "pdbox" a případné "pdbox--loading"
+		// odstraníme všechny class krom "pdbox" a případné "pdbox--loading"
 			.removeClass(function(i, className) {
 				var list = className.split(' ');
 				return list.filter(function(val){
@@ -350,7 +362,7 @@ $.pdBox = (function () {
 			this.window.title.hide();
 		}
 
-		this.dispatchEvent('load', {content: html});
+		this.dispatchEvent('load', {element: $el, content: html});
 	};
 
 	PdBox.prototype.openUrl = function (href, $el) {
@@ -361,7 +373,7 @@ $.pdBox = (function () {
 			success: $.proxy(function (content) {
 				this.window.descWrap.show();
 				this.window.desc.html(content);
-				this.dispatchEvent('load', {content: content});
+				this.dispatchEvent('load', {element: $el, content: content});
 			}, this)
 		});
 	};
@@ -552,7 +564,7 @@ $.pdBox = (function () {
 			}
 
 			if (e.type === 'load') {
-				box.dispatchEvent('load', {content: preloader});
+				box.dispatchEvent('load', {element: $el, content: preloader});
 			}
 		});