Skip to content

Commit

Permalink
update all uikit tooltips to use v-oc-tooltip
Browse files Browse the repository at this point in the history
bump ods
  • Loading branch information
fschade authored and pascalwengerter committed May 19, 2021
1 parent d927513 commit fea7fe2
Show file tree
Hide file tree
Showing 12 changed files with 30 additions and 18 deletions.
10 changes: 10 additions & 0 deletions changelog/unreleased/enhancement-oc-tooltip
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
Enhancement: OcTooltip

We've changed the tooltip implementation to use oc-tooltip directive from ODS instead of uikit's.

https://github.com/owncloud/web/pull/5055
https://github.com/owncloud/web/issues/4654
https://github.com/owncloud/web/issues/2623
https://github.com/owncloud/web/issues/4597
https://github.com/owncloud/web/issues/4332
https://github.com/owncloud/web/issues/4300
2 changes: 1 addition & 1 deletion packages/web-app-files/src/components/AppBar.vue
Original file line number Diff line number Diff line change
Expand Up @@ -27,9 +27,9 @@
<oc-button
id="new-file-menu-btn"
key="new-file-menu-btn-enabled"
v-oc-tooltip="_cannotCreateDialogText"
variation="primary"
appearance="filled"
:uk-tooltip="_cannotCreateDialogText"
:disabled="isNewBtnDisabled"
>
<oc-icon name="add" />
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -66,9 +66,9 @@
<li v-if="$_reshareInformation" class="oc-py-rm">
<oc-tag
:id="$_resharerToggleId"
v-oc-tooltip="$gettext('Show resharer details')"
class="files-collaborators-collaborator-reshare-information"
type="button"
:uk-tooltip="$gettext('Show resharer details')"
>
<oc-icon name="repeat" />
<translate :translate-params="{ resharer: $_reshareInformation }">
Expand Down Expand Up @@ -133,10 +133,10 @@
</li>
<li v-if="isIndirectShare" class="oc-py-rm">
<oc-tag
v-oc-tooltip="viaTooltip"
type="router-link"
class="files-collaborators-collaborator-follow-via"
:to="viaRouterParams"
:uk-tooltip="viaTooltip"
>
<oc-icon name="exit_to_app" />
<span
Expand All @@ -152,8 +152,8 @@
<div class="uk-flex uk-flex-nowrap uk-flex-middle">
<oc-button
v-if="$_editButtonVisible"
v-oc-tooltip="editShareHint"
:aria-label="editShareHint"
:uk-tooltip="editShareHint"
appearance="raw"
class="files-collaborators-collaborator-edit"
@click="$emit('onEdit', collaborator)"
Expand All @@ -163,8 +163,8 @@
<div>
<oc-button
v-if="$_deleteButtonVisible"
v-oc-tooltip="deleteShareHint"
:aria-label="deleteShareHint"
:uk-tooltip="deleteShareHint"
appearance="raw"
class="files-collaborators-collaborator-delete"
@click="$_removeShare"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,8 @@
<div
v-if="canResetExpirationDate"
id="files-collaborators-collaborator-expiration-delete"
v-oc-tooltip="expirationDateRemoveTooltip"
class="uk-position-small uk-position-center-right oc-cursor-pointer"
:uk-tooltip="expirationDateRemoveTooltip"
uk-close
@click="resetExpirationDate"
/>
Expand Down
4 changes: 2 additions & 2 deletions packages/web-app-files/src/components/FileInfoVersions.vue
Original file line number Diff line number Diff line change
Expand Up @@ -16,9 +16,9 @@
<oc-td width="shrink">
<div class="uk-button-group">
<oc-button
v-oc-tooltip="$gettext('Restore older version')"
appearance="raw"
:aria-label="$gettext('Restore older version')"
:uk-tooltip="$gettext('Restore older version')"
@click="revertVersion(item)"
>
<oc-icon name="restore" />
Expand All @@ -28,9 +28,9 @@
<oc-td width="shrink">
<div class="uk-button-group">
<oc-button
v-oc-tooltip="$gettext('Download older version')"
appearance="raw"
:aria-label="$gettext('Download older version')"
:uk-tooltip="$gettext('Download older version')"
@click="downloadVersion(item)"
>
<oc-icon name="cloud_download" />
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,8 @@
<oc-button
v-for="action in filteredActions"
:key="action.label"
v-oc-tooltip="$gettext(action.label)"
:aria-label="$gettext(action.label)"
:uk-tooltip="$gettext(action.label)"
appearance="raw"
class="oc-mr-xs"
:class="`files-quick-action-${action.id}`"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,17 +3,17 @@
<oc-spinner v-if="removalInProgress" :aria-label="$gettext('Removing public link')" />
<template v-else>
<oc-button
v-oc-tooltip="editButtonLabel"
:aria-label="editButtonLabel"
:uk-tooltip="editButtonLabel"
appearance="raw"
class="oc-files-file-link-edit oc-mr-xs"
@click="editLink"
>
<oc-icon name="edit" />
</oc-button>
<oc-button
v-oc-tooltip="deleteButtonLabel"
:aria-label="deleteButtonLabel"
:uk-tooltip="deleteButtonLabel"
appearance="raw"
class="oc-files-file-link-delete"
@click="$_removeLink"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,8 @@
v-text="link.url"
/>
<oc-button
v-oc-tooltip="copyLabel"
:aria-label="copyLabel"
:uk-tooltip="copyLabel"
appearance="raw"
class="oc-files-file-link-copy-url"
>
Expand Down Expand Up @@ -57,10 +57,10 @@
</div>
<div v-if="link.indirect">
<oc-tag
v-oc-tooltip="viaTooltip"
type="router-link"
class="oc-files-file-link-via"
:to="viaRouterParams"
:uk-tooltip="viaTooltip"
>
<oc-icon name="exit_to_app" />
<span class="uk-text-truncate files-file-links-link-via-label" v-text="viaLabel" />
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
<p v-translate class="oc-text-muted oc-my-rm">Only invited people can use this link.</p>
<div class="uk-width-1-1 uk-flex uk-flex-middle">
<a :href="link" target="_blank" class="uk-text-truncate" v-text="link" />
<oc-button :aria-label="copyLabel" :uk-tooltip="copyLabel" appearance="raw" class="oc-ml-s">
<oc-button v-oc-tooltip="copyLabel" :aria-label="copyLabel" appearance="raw" class="oc-ml-s">
<oc-icon
v-if="linkCopied"
id="files-sidebar-private-link-icon-copied"
Expand Down
2 changes: 1 addition & 1 deletion packages/web-runtime/src/components/ApplicationsMenu.vue
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
class="oc-topbar-menu-burger"
:aria-label="applicationSwitcherLabel"
>
<oc-icon :uk-tooltip="applicationSwitcherLabel" name="apps" class="uk-flex" />
<oc-icon v-oc-tooltip="applicationSwitcherLabel" name="apps" class="uk-flex" />
</oc-button>
<oc-drop
ref="menu"
Expand Down
2 changes: 1 addition & 1 deletion packages/web-runtime/src/components/Notifications.vue
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,9 @@
<div id="oc-notification">
<oc-button
id="oc-notification-bell"
v-oc-tooltip="notificationsLabel"
appearance="raw"
:aria-label="notificationsLabel"
:uk-tooltip="notificationsLabel"
>
<oc-icon class="oc-cursor-pointer uk-flex uk-flex-middle" name="bell" />
</oc-button>
Expand Down
6 changes: 4 additions & 2 deletions packages/web-runtime/src/index.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,8 @@
// --- Styles ---
import 'vue-resize/dist/vue-resize.css'

// --- Libraries and Plugins ---
import Vue from './vue'
import 'vue-resize/dist/vue-resize.css'
import VueResize from 'vue-resize'

// --- Components ---
import App from './App.vue'
Expand All @@ -15,6 +16,7 @@ import store from './store'
import router from './router'

// --- Plugins ----
import VueResize from 'vue-resize'
import VueEvents from 'vue-events'
import VueRouter from 'vue-router'
import VueClipboard from 'vue-clipboard2'
Expand Down

0 comments on commit fea7fe2

Please sign in to comment.