Skip to content

Commit

Permalink
Merge pull request #41481 from nextcloud/enh/beta-improvements
Browse files Browse the repository at this point in the history
Enh/beta improvements
  • Loading branch information
szaimen authored Nov 22, 2023
2 parents 99f9456 + f476220 commit 46b0e1b
Show file tree
Hide file tree
Showing 18 changed files with 43 additions and 29 deletions.
4 changes: 2 additions & 2 deletions apps/comments/src/components/Comment.vue
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@
</div>
</div>
<div id="tab-comments__editor-description" class="comment__editor-description">
{{ t('comments', '"@" for mentions, ":" for emoji, "/" for smart picker') }}
{{ t('comments', '@ for mentions, : for emoji, / for smart picker') }}
</div>
</form>

Expand Down Expand Up @@ -288,7 +288,7 @@ $comment-padding: 10px;
&__side {
display: flex;
align-items: flex-start;
padding-top: 16px;
padding-top: 6px;
}

&__body {
Expand Down
20 changes: 17 additions & 3 deletions apps/files/src/views/Sidebar.vue
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@
tabindex="0"
@close="close"
@update:active="setActiveTab"
@update:starred="toggleStarred"
@[defaultActionListener].stop.prevent="onDefaultAction"
@opening="handleOpening"
@opened="handleOpened"
Expand All @@ -50,6 +49,16 @@

<!-- Actions menu -->
<template v-if="fileInfo" #secondary-actions>
<NcActionButton :close-after-click="true"
@click="toggleStarred(!fileInfo.isFavourited)">
<template v-if="fileInfo.isFavourited" #icon>
<StarOutline :size="20" />
</template>
<template v-else #icon>
<Star :size="20" />
</template>
{{ fileInfo.isFavourited ? t('files', 'Add to favorites') : t('files', 'Remove from favorites') }}
</NcActionButton>
<!-- TODO: create proper api for apps to register actions
And inject themselves here. -->
<NcActionButton v-if="isSystemTagsEnabled"
Expand Down Expand Up @@ -98,6 +107,9 @@ import $ from 'jquery'
import axios from '@nextcloud/axios'
import moment from '@nextcloud/moment'

import Star from 'vue-material-design-icons/Star.vue'
import StarOutline from 'vue-material-design-icons/StarOutline.vue'

import NcAppSidebar from '@nextcloud/vue/dist/Components/NcAppSidebar.js'
import NcActionButton from '@nextcloud/vue/dist/Components/NcActionButton.js'
import NcEmptyContent from '@nextcloud/vue/dist/Components/NcEmptyContent.js'
Expand All @@ -117,6 +129,8 @@ export default {
NcEmptyContent,
SidebarTab,
SystemTags,
Star,
StarOutline,
},

data() {
Expand Down Expand Up @@ -189,7 +203,8 @@ export default {
* @return {string}
*/
subtitle() {
return `${this.size}, ${this.time}`
const starredIndicator = this.fileInfo.isFavourited ? '★ ' : ''
return `${starredIndicator} ${this.size}, ${this.time}`
},

/**
Expand Down Expand Up @@ -246,7 +261,6 @@ export default {
},
compact: this.hasLowHeight || !this.fileInfo.hasPreview || this.isFullScreen,
loading: this.loading,
starred: this.fileInfo.isFavourited,
subname: this.subtitle,
subtitle: this.fullTime,
name: this.fileInfo.name,
Expand Down
4 changes: 2 additions & 2 deletions dist/7255-7255.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/7255-7255.js.map

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions dist/7856-7856.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/7856-7856.js.map

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions dist/comments-comments-app.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/comments-comments-app.js.map

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions dist/comments-comments-tab.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/comments-comments-tab.js.map

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions dist/core-common.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/core-common.js.map

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions dist/files-sidebar.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/files-sidebar.js.map

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions dist/files_sharing-files_sharing_tab.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/files_sharing-files_sharing_tab.js.map

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions dist/weather_status-weather-status.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/weather_status-weather-status.js.map

Large diffs are not rendered by default.

0 comments on commit 46b0e1b

Please sign in to comment.