@tugitark/vue-widget 1.1.1 → 1.2.0

package/index.js

@@ -49,23 +49,22 @@ export default {
 	methods: {
 		onNotification_(hasNotification) {
 			if (this.jwtFn == null && this.user == null) {
-				if (this.firstTime_) {
+				if (!this.firstTime_) {
 					// Replicate FAQ internal behaviour at a per-component level.
-					this.onNotification?.(hasNotification);
+					return;
 				}
-			} else {
-				this.onNotification?.(hasNotification);
 			}
+			this.onNotification && this.onNotification(hasNotification);
 		},
 		onReady_() {
 			if (this.firstTime_) {
 				this.firstTime_ = false;
-				this.onReady?.();
+				this.onReady && this.onReady();
 			}
 		},
-		onOpened_() { return this.onOpened?.(); },
-		onClosed_() { return this.onClosed?.(); },
-		onError_(e) { return this.onError?.(e); },
+		onOpened_() { this.onOpened && this.onOpened(); },
+		onClosed_() { this.onClosed && this.onClosed(); },
+		onError_(e) { this.onError && this.onError(e); },
 	},
 	watc() {
 		return render(this.props_);

package/index.md

@@ -158,7 +158,7 @@ The following optional properties are shared with the full version:
  Notes
 -------
 
-While there can be only one widget on the page there can be multiple `<TugiWidget>` components at once.  Which one will be used is non-deterministic - even if you have one `<TugiWidget>` with `visible={true}` and another one with `visible={false}` this will not guarantee that the visible one is the one that gets shown, the invisible one may be "shown", resulting in neither being seen.  The one selected is based on internal render and update orders.  Further, having two at once may result in the widget being constantly unloaded and reloaded even when nothing has changed as the underlying code flips repeatedly between them.  If you have two versions of the widget shown in different circumstances then make sure that the whole component is conditional:
+While there can be only one widget on the page there can be multiple `<TugiWidget>` components at once.  Which one will be used is non-deterministic - even if you have one `<TugiWidget>` with `:visible="true"` and another one with `:visible="false"` this will not guarantee that the visible one is the one that gets shown, the invisible one may be "shown", resulting in neither being seen.  The one selected is based on internal render and update orders.  Further, having two at once may result in the widget being constantly unloaded and reloaded even when nothing has changed as the underlying code flips repeatedly between them.  If you have two versions of the widget shown in different circumstances then make sure that the whole component is conditional:
 
 
 ```javascript
@@ -190,6 +190,7 @@ While there can be only one widget on the page there can be multiple `<TugiWidge
 			homePageBackgroundColor: 'hsla(36, 100%, 37%, 0.2)',
 			homePageTextColor: '#2c3e50',
 		}"
-	/></template>
+	/>
+</template>
 ```
 

package/package.json

@@ -1,13 +1,16 @@
 {
 	"name": "@tugitark/vue-widget",
-	"version": "1.1.1",
+	"version": "1.2.0",
 	"description": "Wraps the tugitark widget in a Vue component.",
+	"scripts": {
+		"release": "npm publish --access public"
+	},
 	"files": [
 		"index.js",
 		"index.md"
 	],
 	"dependencies": {
-		"@tugitark/declarative-widget": "^1.0.1"
+		"@tugitark/declarative-widget": "^1.0.2"
 	},
 	"peerDependencies": {
 		"vue": ">=0.0.0"