Skip to content

Commit

Permalink
Merge pull request #44059 from nextcloud/fix/settings-apps-level
Browse files Browse the repository at this point in the history
fix(settings): Drop deprecated code from app level badge
  • Loading branch information
skjnldsv authored Mar 7, 2024
2 parents bfcbffa + e39e564 commit be3449c
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 13 deletions.
8 changes: 1 addition & 7 deletions apps/settings/src/components/AppList/AppLevelBadge.vue
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
class="app-level-badge"
:class="{ 'app-level-badge--supported': isSupported }"
:title="badgeTitle">
<NcIconSvgWrapper :path="badgeIcon" :size="20" />
<NcIconSvgWrapper :path="badgeIcon" :size="20" inline />
{{ badgeText }}
</span>
</template>
Expand Down Expand Up @@ -69,11 +69,5 @@ const badgeTitle = computed(() => isSupported.value
border-color: var(--color-success);
color: var(--color-success);
}
// Fix the svg wrapper TODO: Remove with @nextcloud/vue 8.8.0 release
:deep(.icon-vue) {
min-width: unset;
min-height: unset;
}
}
</style>
4 changes: 2 additions & 2 deletions dist/settings-apps-view-4529.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/settings-apps-view-4529.js.map

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions dist/settings-vue-settings-apps-users-management.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/settings-vue-settings-apps-users-management.js.map

Large diffs are not rendered by default.

0 comments on commit be3449c

Please sign in to comment.