Skip to content

Commit

Permalink
Merge pull request #1092 from nextcloud/bugfix/noid/open-deck-links-i…
Browse files Browse the repository at this point in the history
…n-same-tab

Open the deck links in the same tab
  • Loading branch information
nickvergessen authored Oct 15, 2021
2 parents 6d47210 + 5514d81 commit 519c37c
Show file tree
Hide file tree
Showing 12 changed files with 32 additions and 16 deletions.
4 changes: 2 additions & 2 deletions js/notifications-main.js

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions js/notifications-main.js.LICENSE.txt
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@
/**
* @copyright Copyright (c) 2018 Joas Schilling <coding@schilljs.com>
*
* @license GNU AGPL version 3 or any later version
* @license AGPL-3.0-or-later
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as
Expand Down Expand Up @@ -220,7 +220,7 @@
/**
* @copyright Copyright (c) 2020 Joas Schilling <coding@schilljs.com>
*
* @license GNU AGPL version 3 or any later version
* @license AGPL-3.0-or-later
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as
Expand Down
2 changes: 1 addition & 1 deletion js/notifications-main.js.map

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion js/notifications-settings.js.LICENSE.txt
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@
/**
* @copyright Copyright (c) 2021 Julien Barnoin <julien@barnoin.com>
*
* @license GNU AGPL version 3 or any later version
* @license AGPL-3.0-or-later
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as
Expand Down
2 changes: 1 addition & 1 deletion js/notifications-settings.js.map

Large diffs are not rendered by default.

10 changes: 6 additions & 4 deletions src/App.vue
Original file line number Diff line number Diff line change
Expand Up @@ -245,7 +245,8 @@ export default {
/**
* Update the title to show * if there are new notifications
* @param {Object} notifications The list of notifications
*
* @param {object} notifications The list of notifications
*/
_updateDocTitleOnNewNotifications(notifications) {
if (notifications.length > this._oldcount) {
Expand All @@ -271,8 +272,8 @@ export default {
},
/**
* Performs the AJAX request to retrieve the notifications
*/
* Performs the AJAX request to retrieve the notifications
*/
async _fetch() {
const response = await getNotificationsData(this.tabId, this.lastETag, !this.backgroundFetching)
Expand Down Expand Up @@ -339,7 +340,8 @@ export default {
/**
* The app was disabled or has no notifiers, so we can stop polling
* And hide the UI as well
* @param {Boolean} temporary If false, the notification bell will be hidden
*
* @param {boolean} temporary If false, the notification bell will be hidden
*/
_shutDownNotifications(temporary) {
console.debug('Shutting down notifications ' + ((temporary) ? 'temporary' : 'bye'))
Expand Down
1 change: 1 addition & 0 deletions src/Components/Notification.vue
Original file line number Diff line number Diff line change
Expand Up @@ -266,6 +266,7 @@ export default {
/**
* Create a browser notification
*
* @see https://developer.mozilla.org/en/docs/Web/API/notification
*/
_createWebNotification() {
Expand Down
15 changes: 14 additions & 1 deletion src/Components/Parameters/DefaultParameter.vue
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,11 @@
-->

<template>
<a v-if="link"
<a v-if="hasInternalLink"
:href="link">
<strong>{{ name }}</strong>
</a>
<a v-else-if="link"
:href="link"
class="external"
target="_blank"
Expand Down Expand Up @@ -51,6 +55,15 @@ export default {
default: '',
},
},
computed: {
hasInternalLink() {
return this.link && (
this.type === 'deck-board'
|| this.type === 'deck-card'
)
},
},
}
</script>

Expand Down
2 changes: 1 addition & 1 deletion src/main.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
/**
* @copyright Copyright (c) 2018 Joas Schilling <coding@schilljs.com>
*
* @license GNU AGPL version 3 or any later version
* @license AGPL-3.0-or-later
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as
Expand Down
2 changes: 1 addition & 1 deletion src/services/BrowserStorage.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
/**
* @copyright Copyright (c) 2020 Joas Schilling <coding@schilljs.com>
*
* @license GNU AGPL version 3 or any later version
* @license AGPL-3.0-or-later
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as
Expand Down
2 changes: 1 addition & 1 deletion src/services/notificationsService.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
/**
* @copyright Copyright (c) 2020 Joas Schilling <coding@schilljs.com>
*
* @license GNU AGPL version 3 or any later version
* @license AGPL-3.0-or-later
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as
Expand Down
2 changes: 1 addition & 1 deletion src/settings.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
/**
* @copyright Copyright (c) 2021 Julien Barnoin <julien@barnoin.com>
*
* @license GNU AGPL version 3 or any later version
* @license AGPL-3.0-or-later
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as
Expand Down

0 comments on commit 519c37c

Please sign in to comment.