diff --git a/packages/design-system/src/components/OcStatusIndicators/OcStatusIndicators.vue b/packages/design-system/src/components/OcStatusIndicators/OcStatusIndicators.vue index 8c3812a4204..3c8271318d4 100644 --- a/packages/design-system/src/components/OcStatusIndicators/OcStatusIndicators.vue +++ b/packages/design-system/src/components/OcStatusIndicators/OcStatusIndicators.vue @@ -10,11 +10,12 @@ :aria-label="$gettext(indicator.label)" :aria-describedby="getIndicatorDescriptionId(indicator)" appearance="raw-inverse" + variation="primary" :data-testid="indicator.id" :data-test-indicator-type="indicator.type" @click="indicator.handler(resource, indicator.target, $router)" > - + diff --git a/packages/web-app-files/src/components/SideBar/Shares/Collaborators/RoleDropdown.vue b/packages/web-app-files/src/components/SideBar/Shares/Collaborators/RoleDropdown.vue index 4b4daa9c117..ca559011fbc 100644 --- a/packages/web-app-files/src/components/SideBar/Shares/Collaborators/RoleDropdown.vue +++ b/packages/web-app-files/src/components/SideBar/Shares/Collaborators/RoleDropdown.vue @@ -30,25 +30,25 @@ - + - + @@ -272,7 +272,7 @@ export default defineComponent({ this.publishChange() }, - isSelectedRole(role) { + isSelectedRole(role: ShareRole) { return this.selectedRole.name === role.name }, @@ -367,15 +367,6 @@ export default defineComponent({ width: 400px; &-list { - &:hover .files-recipient-role-drop-btn.selected:not(:hover), - &:focus .files-recipient-role-drop-btn.selected:not(:focus) { - color: var(--oc-color-swatch-passive-default); - - ::v-deep .oc-icon > svg { - fill: var(--oc-color-swatch-passive-default); - } - } - li { margin: var(--oc-space-xsmall) 0; } @@ -388,17 +379,8 @@ export default defineComponent({ &:hover, &:focus { background-color: var(--oc-color-background-hover); - color: var(--oc-color-swatch-passive-default); text-decoration: none; } - - &.selected { - color: var(--oc-color-swatch-passive-contrast) !important; - - ::v-deep .oc-icon > svg { - fill: var(--oc-color-swatch-passive-contrast) !important; - } - } } } diff --git a/packages/web-app-files/src/components/SideBar/Shares/Links/DetailsAndEdit.vue b/packages/web-app-files/src/components/SideBar/Shares/Links/DetailsAndEdit.vue index 9c31c14caa2..f44d9062ec6 100644 --- a/packages/web-app-files/src/components/SideBar/Shares/Links/DetailsAndEdit.vue +++ b/packages/web-app-files/src/components/SideBar/Shares/Links/DetailsAndEdit.vue @@ -20,18 +20,15 @@ mode="click" > -
  • +
  • - + - +
  • @@ -402,6 +399,10 @@ export default defineComponent({ methods: { ...mapActions(['createModal', 'hideModal', 'setModalInputErrorMessage']), + isSelectedRole(role: ShareRole) { + return this.link.permissions === role.bitmask(false) + }, + updateLink({ link = this.link, dropRef = this.$refs.editPublicLinkDropdown, @@ -503,17 +504,8 @@ export default defineComponent({ &:hover, &:focus { background-color: var(--oc-color-background-hover); - color: var(--oc-color-swatch-passive-default); text-decoration: none; } - - &.selected { - color: var(--oc-color-swatch-passive-contrast) !important; - - ::v-deep .oc-icon > svg { - fill: var(--oc-color-swatch-passive-contrast) !important; - } - } } } diff --git a/packages/web-runtime/src/components/SidebarNav/SidebarNavItem.vue b/packages/web-runtime/src/components/SidebarNav/SidebarNavItem.vue index 25d32e5ca18..61d90ada28b 100644 --- a/packages/web-runtime/src/components/SidebarNav/SidebarNavItem.vue +++ b/packages/web-runtime/src/components/SidebarNav/SidebarNavItem.vue @@ -15,7 +15,7 @@ :data-nav-name="navName" > - + {{ tag }} diff --git a/packages/web-runtime/src/components/Topbar/ApplicationsMenu.vue b/packages/web-runtime/src/components/Topbar/ApplicationsMenu.vue index a1c741d8d49..bf080830c3e 100644 --- a/packages/web-runtime/src/components/Topbar/ApplicationsMenu.vue +++ b/packages/web-runtime/src/components/Topbar/ApplicationsMenu.vue @@ -36,10 +36,10 @@ :class="{ 'oc-background-primary-gradient router-link-active': n.active }" > - + - +
    @@ -104,7 +104,6 @@ export default defineComponent({ &:focus, &:hover { background-color: var(--oc-color-background-hover); - color: var(--oc-color-swatch-passive-default); text-decoration: none; } @@ -121,13 +120,5 @@ export default defineComponent({ right: 1rem; } } - - a.router-link-active, - button.router-link-active { - &:focus, - &:hover { - color: var(--oc-color-swatch-passive-contrast); - } - } } diff --git a/packages/web-runtime/src/components/Topbar/ThemeSwitcher.vue b/packages/web-runtime/src/components/Topbar/ThemeSwitcher.vue index 356feadac65..d41b0794c37 100644 --- a/packages/web-runtime/src/components/Topbar/ThemeSwitcher.vue +++ b/packages/web-runtime/src/components/Topbar/ThemeSwitcher.vue @@ -8,7 +8,7 @@ @click="toggleTheme" > - +