Skip to content

Commit

Permalink
Merge pull request #40938 from nextcloud/fix/a11y/change-scope-actions
Browse files Browse the repository at this point in the history
fix(settings): Account property scope actions
  • Loading branch information
Pytal authored Oct 24, 2023
2 parents d772537 + 5a562a5 commit 450985f
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 18 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -34,8 +34,7 @@
:is-supported-scope="supportedScopes.includes(federationScope.name)"
:name="federationScope.name"
:tooltip-disabled="federationScope.tooltipDisabled"
:tooltip="federationScope.tooltip"
:aria-label="federationScope.tooltip" />
:tooltip="federationScope.tooltip" />
</NcActions>
</template>

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,13 +21,12 @@
-->

<template>
<NcActionButton :aria-label="isSupportedScope ? tooltip : tooltipDisabled"
class="federation-actions__btn"
<NcActionButton class="federation-actions__btn"
:class="{ 'federation-actions__btn--active': activeScope === name }"
:close-after-click="true"
:disabled="!isSupportedScope"
:icon="iconClass"
:title="displayName"
:name="displayName"
@click.stop.prevent="updateScope">
{{ isSupportedScope ? tooltip : tooltipDisabled }}
</NcActionButton>
Expand Down Expand Up @@ -87,18 +86,9 @@ export default {
</script>

<style lang="scss" scoped>
.federation-actions__btn {
&::v-deep p {
width: 150px !important;
padding: 8px 0 !important;
color: var(--color-main-text) !important;
font-size: 12.8px !important;
line-height: 1.5em !important;
}
}
.federation-actions__btn--active {
background-color: var(--color-primary-element-light) !important;
box-shadow: inset 2px 0 var(--color-primary-element) !important;
border-radius: 0px !important;
}
</style>
4 changes: 2 additions & 2 deletions dist/settings-vue-settings-personal-info.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/settings-vue-settings-personal-info.js.map

Large diffs are not rendered by default.

0 comments on commit 450985f

Please sign in to comment.