Fix asterix conflict with talk

Signed-off-by: Joas Schilling <coding@schilljs.com>
nextcloud · Jun 21, 2021 · 2287957 · 2287957
1 parent f2d1a05
commit 2287957
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 14 deletions.
diff --git a/js/notifications-main.js b/js/notifications-main.js
diff --git a/js/notifications-main.js.map b/js/notifications-main.js.map
diff --git a/src/App.vue b/src/App.vue
@@ -248,17 +248,12 @@ export default {
 		 * @param {Object} notifications The list of notifications
 		 */
 		_updateDocTitleOnNewNotifications(notifications) {
-			if (notifications.length < this._oldcount) {
-				this._restoreTitle()
-				this._oldcount = notifications.length
-			} else if (notifications.length > this._oldcount) {
+			if (notifications.length > this._oldcount) {
 				this._oldcount = notifications.length
 				if (this.backgroundFetching && document.hidden) {
 					// If we didn't already highlight, store the title so we can restore on tab-view
-					if (self._setTitle !== document.title) {
-						self._storedTitle = document.title
-						self._setTitle = '* ' + document.title
-						document.title = self._setTitle
+					if (!document.title.startsWith('* ')) {
+						document.title = '* ' + document.title
 					}
 				}
 			}
@@ -270,9 +265,8 @@ export default {
 		 * the Talk might have altered it.
 		 */
 		_restoreTitle() {
-			if (self._setTitle === document.title) {
-				document.title = self._storedTitle
-				self._setTitle = null
+			if (document.title.startsWith('* ')) {
+				document.title = document.title.substring(2)
 			}
 		},