Do not allow disabled links (#557)

Do not allow disabled links
nextcloud-libraries · Aug 29, 2019 · e2e9d8d · e2e9d8d
2 parents 907bbbd + f860d8e
commit e2e9d8d
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 16 deletions.
diff --git a/src/components/ActionButton/ActionButton.vue b/src/components/ActionButton/ActionButton.vue
@@ -28,6 +28,7 @@ This component is made to be used inside of the [Actions](#Actions) component sl
 	<Actions>
 		<ActionButton icon="icon-delete" @click="alert('Delete')">Delete</ActionButton>
 		<ActionButton icon="icon-delete" @click="alert('Delete')">Delete</ActionButton>
+		<ActionButton icon="icon-delete" :disabled="true" @click="alert('Disabled')">Disabled button</ActionButton>
 	</Actions>
 ```
 

diff --git a/src/components/ActionLink/ActionLink.vue b/src/components/ActionLink/ActionLink.vue
@@ -37,7 +37,7 @@ This component is made to be used inside of the [Actions](#Actions) component sl
 			:download="download"
 			:href="href"
 			:target="target"
-			:class="['action-link', { focusable: isFocusable }]"
+			class="action-link focusable"
 			rel="noreferrer noopener"
 			@click="onClick">
 
@@ -113,13 +113,6 @@ export default {
 			validator: value => {
 				return ['_blank', '_self', '_parent', '_top'].indexOf(value) > -1
 			}
-		},
-		/**
-		 * Disabled state of the link
-		 */
-		disabled: {
-			type: Boolean,
-			default: false
 		}
 	},
 	computed: {

diff --git a/src/components/ActionRouter/ActionRouter.vue b/src/components/ActionRouter/ActionRouter.vue
@@ -25,7 +25,7 @@
 	<li :class="{ 'action--disabled': disabled }">
 		<router-link :to="to"
 			:exact="exact"
-			:class="[ 'action-router', { focusable : isFocusable }]"
+			class="action-router focusable"
 			rel="noreferrer noopener">
 			<!-- icon -->
 			<span :class="[isIconUrl ? 'action-router__icon--url' : icon]"
@@ -81,13 +81,6 @@ export default {
 		exact: {
 			type: Boolean,
 			default: false
-		},
-		/**
-		 * Disabled state of the router
-		 */
-		disabled: {
-			type: Boolean,
-			default: false
 		}
 	},
 	computed: {