Merge pull request #41481 from nextcloud/enh/beta-improvements

Enh/beta improvements
This commit is contained in:
Simon L 2023-11-22 16:29:53 +01:00 committed by GitHub
commit 46b0e1bef1
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
18 changed files with 43 additions and 29 deletions

View file

@ -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>
@ -288,7 +288,7 @@ $comment-padding: 10px;
&__side {
display: flex;
align-items: flex-start;
padding-top: 16px;
padding-top: 6px;
}
&__body {

View file

@ -28,7 +28,6 @@
tabindex="0"
@close="close"
@update:active="setActiveTab"
@update:starred="toggleStarred"
@[defaultActionListener].stop.prevent="onDefaultAction"
@opening="handleOpening"
@opened="handleOpened"
@ -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"
@ -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'
@ -117,6 +129,8 @@ export default {
NcEmptyContent,
SidebarTab,
SystemTags,
Star,
StarOutline,
},
data() {
@ -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}`
},
/**
@ -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,

4
dist/7255-7255.js vendored

File diff suppressed because one or more lines are too long

File diff suppressed because one or more lines are too long

4
dist/7856-7856.js vendored

File diff suppressed because one or more lines are too long

File diff suppressed because one or more lines are too long

File diff suppressed because one or more lines are too long

File diff suppressed because one or more lines are too long

File diff suppressed because one or more lines are too long

File diff suppressed because one or more lines are too long

4
dist/core-common.js vendored

File diff suppressed because one or more lines are too long

File diff suppressed because one or more lines are too long

File diff suppressed because one or more lines are too long

File diff suppressed because one or more lines are too long

File diff suppressed because one or more lines are too long

File diff suppressed because one or more lines are too long

File diff suppressed because one or more lines are too long

File diff suppressed because one or more lines are too long