Add full width for mailing list

Signed-off-by: greta <[email protected]>

lib/Controller/PageController.php

@@ -206,6 +206,7 @@ public function index(): TemplateResponse {
 				'attachment-size-limit' => $this->config->getSystemValue('app.mail.attachment-size-limit', 0),
 				'app-version' => $this->config->getAppValue('mail', 'installed_version'),
 				'external-avatars' => $this->preferences->getPreference($this->currentUserId, 'external-avatars', 'true'),
+				'layout-mode' => $this->preferences->getPreference($this->currentUserId, 'layout-mode', 'vertical-split'),
 				'reply-mode' => $this->preferences->getPreference($this->currentUserId, 'reply-mode', 'top'),
 				'collect-data' => $this->preferences->getPreference($this->currentUserId, 'collect-data', 'true'),
 				'search-priority-body' => $this->preferences->getPreference($this->currentUserId, 'search-priority-body', 'false'),

src/components/AppSettingsMenu.vue

@@ -4,6 +4,28 @@
 			:name="t('mail', 'Mail settings')"
 			:show-navigation="true"
 			:open.sync="showSettings">
+			<NcAppSettingsSection id="mail-list-view" :name="t('mail', 'Layout')">
+				<NcCheckboxRadioSwitch value="no-split"
+					name="mail-layout"
+					type="radio"
+					:checked="layoutMode"
+					@update:checked="setLayout">
+					<template #icon>
+						<CompactMode :size="20" />
+					</template>
+					{{ t('mail', 'List') }}
+				</NcCheckboxRadioSwitch>
+				<NcCheckboxRadioSwitch value="vertical-split"
+					name="mail-layout"
+					type="radio"
+					:checked="layoutMode"
+					@update:checked="setLayout">
+					<template #icon>
+						<VerticalSplit :size="20" />
+					</template>
+					{{ t('mail', 'Vertical split') }}
+				</NcCheckboxRadioSwitch>
+			</NcAppSettingsSection>
 			<NcAppSettingsSection id="account-settings" :name="t('mail', 'Account creation')">
 				<NcButton v-if="allowNewMailAccounts"
 					type="primary"
@@ -120,22 +142,22 @@
 			</NcAppSettingsSection>
 			<NcAppSettingsSection id="sorting-settings" :name="t('mail', 'Sorting')">
 				<div class="sorting">
-					<CheckboxRadioSwitch class="sorting__switch"
+					<NcCheckboxRadioSwitch class="sorting__switch"
 						:checked="sortOrder"
 						value="newest"
 						name="order_radio"
 						type="radio"
 						@update:checked="onSortByDate">
 						{{ t('mail', 'Newest') }}
-					</CheckboxRadioSwitch>
-					<CheckboxRadioSwitch class="sorting__switch"
+					</NcCheckboxRadioSwitch>
+					<NcCheckboxRadioSwitch class="sorting__switch"
 						:checked="sortOrder"
 						value="oldest"
 						name="order_radio"
 						type="radio"
 						@update:checked="onSortByDate">
 						{{ t('mail', 'Oldest') }}
-					</CheckboxRadioSwitch>
+					</NcCheckboxRadioSwitch>
 				</div>
 			</NcAppSettingsSection>
 			<NcAppSettingsSection id="mailvelope-settings" :name="t('mail', 'Mailvelope')">
@@ -202,12 +224,14 @@
 <script>
 import { generateUrl } from '@nextcloud/router'
 import { showError } from '@nextcloud/dialogs'
+import CompactMode from 'vue-material-design-icons/ReorderHorizontal.vue'
 
-import { NcAppSettingsSection, NcAppSettingsDialog, NcButton, NcLoadingIcon as IconLoading, NcCheckboxRadioSwitch as CheckboxRadioSwitch } from '@nextcloud/vue'
+import { NcAppSettingsSection, NcAppSettingsDialog, NcButton, NcLoadingIcon as IconLoading, NcCheckboxRadioSwitch } from '@nextcloud/vue'
 
 import IconAdd from 'vue-material-design-icons/Plus.vue'
 import IconEmail from 'vue-material-design-icons/Email.vue'
 import IconLock from 'vue-material-design-icons/Lock.vue'
+import VerticalSplit from 'vue-material-design-icons/FormatColumns.vue'
 import Logger from '../logger.js'
 import SmimeCertificateModal from './smime/SmimeCertificateModal.vue'
 import TrustedSenders from './TrustedSenders.vue'
@@ -222,9 +246,11 @@ export default {
 		IconLoading,
 		IconLock,
 		SmimeCertificateModal,
-		CheckboxRadioSwitch,
+		NcCheckboxRadioSwitch,
 		NcAppSettingsDialog,
 		NcAppSettingsSection,
+		CompactMode,
+		VerticalSplit,
 	},
 	props: {
 		open: {
@@ -270,6 +296,9 @@ export default {
 		allowNewMailAccounts() {
 			return this.$store.getters.getPreference('allow-new-accounts', true)
 		},
+		layoutMode() {
+			return this.$store.getters.getPreference('layout-mode', 'vertical-split')
+		},
 	},
 	watch: {
 		showSettings(value) {
@@ -287,6 +316,17 @@ export default {
 		this.sortOrder = this.$store.getters.getPreference('sort-order', 'newest')
 	},
 	methods: {
+		setLayout(value) {
+			this.$store
+				.dispatch('savePreference', {
+					key: 'layout-mode',
+					value,
+				})
+
+				.catch((error) => Logger.error('could not save preferences', { error }))
+				.then(() => {
+				})
+		},
 		async onOpen() {
 			this.showSettings = true
 		},
@@ -465,4 +505,7 @@ p.app-settings {
 	margin-bottom: 6px;
 	color: var(--color-text-maxcontrast);
 }
+.app-settings-section {
+	list-style: none;
+}
 </style>

src/components/Envelope.vue

@@ -14,6 +14,7 @@
 		:data-envelope-id="data.databaseId"
 		:name="addresses"
 		:details="formatted()"
+		:one-line="oneLineLayout"
 		@click="onClick"
 		@click.ctrl.prevent="toggleSelected"
 		@update:menuOpen="closeMoreAndSnoozeOptions">
@@ -22,16 +23,19 @@
 				fill-color="#f9cf3d"
 				:size="18"
 				class="app-content-list-item-star favorite-icon-style"
+				:class="{ 'one-line': oneLineLayout, 'favorite-icon-style': !oneLineLayout }"
 				:data-starred="data.flags.flagged ? 'true' : 'false'"
 				@click.prevent="hasWriteAcl ? onToggleFlagged() : false" />
 			<div v-if="isImportant"
 				class="app-content-list-item-star svg icon-important"
+				:class="{ 'important-one-line': oneLineLayout, 'icon-important': !oneLineLayout }"
 				:data-starred="isImportant ? 'true' : 'false'"
 				@click.prevent="hasWriteAcl ? onToggleImportant() : false"
 				v-html="importantSvg" />
 			<JunkIcon v-if="data.flags.$junk"
 				:size="18"
 				class="app-content-list-item-star junk-icon-style"
+				:class="{ 'one-line': oneLineLayout, 'junk-icon-style': !oneLineLayout }"
 				:data-starred="data.flags.$junk ? 'true' : 'false'"
 				@click.prevent="hasWriteAcl ? onToggleJunk() : false" />
 			<div class="app-content-list-item-icon">
@@ -48,23 +52,26 @@
 			</div>
 		</template>
 		<template #subname>
-			<div class="envelope__subtitle">
-				<Reply v-if="data.flags.answered"
-					class="seen-icon-style"
-					:size="18" />
-				<IconAttachment v-if="data.flags.hasAttachments === true"
-					class="attachment-icon-style"
-					:size="18" />
-				<span v-else-if="draft" class="draft">
-					<em>{{ t('mail', 'Draft: ') }}</em>
-				</span>
-				<span class="envelope__subtitle__subject">
-					{{ subjectForSubtitle }}
-				</span>
-			</div>
-			<div v-if="data.encrypted || data.previewText"
-				class="envelope__preview-text">
-				{{ isEncrypted ? t('mail', 'Encrypted message') : data.previewText.trim() }}
+			<div class="line-two"
+				:class="{ 'one-line': oneLineLayout }">
+				<div class="envelope__subtitle">
+					<Reply v-if="data.flags.answered"
+						class="seen-icon-style"
+						:size="18" />
+					<IconAttachment v-if="data.flags.hasAttachments === true"
+						class="attachment-icon-style"
+						:size="18" />
+					<span v-else-if="draft" class="draft">
+						<em>{{ t('mail', 'Draft: ') }}</em>
+					</span>
+					<span class="envelope__subtitle__subject">
+						{{ subjectForSubtitle }}
+					</span>
+				</div>
+				<div v-if="data.encrypted || data.previewText"
+					class="envelope__preview-text">
+					{{ isEncrypted ? t('mail', 'Encrypted message') : data.previewText.trim() }}
+				</div>
 			</div>
 		</template>
 		<template #indicator>
@@ -126,7 +133,7 @@
 						messageLongDate
 					}}
 				</ActionText>
-				<ActionSeparator />
+				<NcActionSeparator />
 				<ActionButton v-if="hasWriteAcl"
 					:close-after-click="true"
 					@click.prevent="onToggleJunk">
@@ -320,7 +327,8 @@ import {
 	NcListItem as ListItem,
 	NcActionButton as ActionButton,
 	NcActionLink as ActionLink,
-	NcActionSeparator as ActionSeparator,
+	NcActionSeparator,
+	NcActionInput,
 	NcActionText as ActionText,
 } from '@nextcloud/vue'
 import AlertOctagonIcon from 'vue-material-design-icons/AlertOctagon.vue'
@@ -364,8 +372,6 @@ import { generateUrl } from '@nextcloud/router'
 import { isPgpText } from '../crypto/pgp.js'
 import { mailboxHasRights } from '../util/acl.js'
 import DownloadIcon from 'vue-material-design-icons/Download.vue'
-import NcActionSeparator from '@nextcloud/vue/dist/Components/NcActionSeparator.js'
-import NcActionInput from '@nextcloud/vue/dist/Components/NcActionInput.js'
 import CalendarClock from 'vue-material-design-icons/CalendarClock.vue'
 import AlarmIcon from 'vue-material-design-icons/Alarm.vue'
 import moment from '@nextcloud/moment'
@@ -403,7 +409,6 @@ export default {
 		IconBullet,
 		Reply,
 		ActionLink,
-		ActionSeparator,
 		ActionText,
 		DownloadIcon,
 		ClockOutlineIcon,
@@ -467,6 +472,9 @@ export default {
 		messageLongDate() {
 			return messageDateTime(new Date(this.data.dateInt))
 		},
+		oneLineLayout() {
+			return this.$store.getters.getPreference('layout-mode', 'vertical-split') === 'no-split'
+		},
 		hasMultipleRecipients() {
 			if (!this.account) {
 				console.error('account is undefined', {
@@ -844,7 +852,6 @@ export default {
 	},
 }
 </script>
-
 <style lang="scss" scoped>
 .mail-message-account-color {
 	position: absolute;
@@ -860,20 +867,25 @@ export default {
 	}
 
 	&__subtitle {
-		display: flex;
-		gap: 4px;
-
+		max-width: 80%;
+		overflow: hidden;
+		text-overflow: ellipsis;
+		white-space: nowrap;
 		&__subject {
 			line-height: 130%;
-			overflow: hidden;
-			text-overflow: ellipsis;
+			&::after {
+				content: '\00B7';
+				margin: 12px;
+			}
 		}
 	}
 	&__preview-text {
+		color: var(--color-text-maxcontrast);
 		white-space: nowrap;
 		overflow: hidden;
 		text-overflow: ellipsis;
 		font-weight: initial;
+		flex: 1 1;
 	}
 }
 
@@ -903,6 +915,9 @@ export default {
 	}
 	}
 }
+.important-one-line.app-content-list-item-star:deep() {
+	top: 1px !important;
+}
 
 .app-content-list-item-select-checkbox {
 	display: inline-block;
@@ -986,7 +1001,7 @@ export default {
 }
 .icon-important.app-content-list-item-star:deep() {
 	position: absolute;
-	top: 14px;
+	top: 23px;
 	z-index: 1;
 }
 .app-content-list-item-star.favorite-icon-style {
@@ -1008,9 +1023,11 @@ export default {
 }
 .seen-icon-style {
 	opacity: .6;
+	display: inline;
 }
 .attachment-icon-style {
 	opacity: .6;
+	display: inline;
 }
 :deep(.list-item__anchor) {
 	margin-top: 6px;
@@ -1019,9 +1036,21 @@ export default {
 :deep(.list-item) {
 	flex-wrap: wrap;
 }
-:deep(.list-item__extra) {
-	margin-top: 9px;
+:deep(.line-two__subtitle) {
+	display: flex;
 	flex-basis: 100%;
 	padding-left: 40px;
+	width: 450px;
+}
+:deep(.line-one__title) {
+	flex-direction: row;
+	display: flex;
+	width: 200px;
+}
+.line-two.one-line {
+	display: flex;
+	overflow: hidden;
+	text-overflow: ellipsis;
+	white-space: nowrap;
 }
 </style>

src/components/EnvelopeList.vue

@@ -320,6 +320,7 @@ export default {
 			showMoveModal: false,
 			showTagModal: false,
 			lastToggledIndex: undefined,
+			defaultView: false,
 		}
 	},
 	computed: {

src/components/Mailbox.vue

@@ -2,6 +2,7 @@
   - @copyright 2020 Christoph Wurst <christoph@winzerhof-wurst.at>
   -
   - @author 2020 Christoph Wurst <[email protected]>
+  - @author Richard Steinmetz <[email protected]>
   -
   - @license AGPL-3.0-or-later
   -
@@ -111,7 +112,7 @@ export default {
 			error: false,
 			refreshing: false,
 			loadingMore: false,
-			loadingEnvelopes: true,
+			loadingEnvelopes: false,
 			loadingCacheInitialization: false,
 			loadMailboxInterval: undefined,
 			expanded: false,
@@ -161,11 +162,14 @@ export default {
 		this.loadMailboxInterval = setInterval(this.loadMailbox, 60000)
 	},
 	async mounted() {
-		this.loadEnvelopes()
-			.then(() => {
-				logger.debug(`syncing mailbox ${this.mailbox.databaseId} (${this.searchQuery}) after mount`)
-				this.sync(false)
-			})
+		if (this.$store.getters.hasFetchedInitialEnvelopes) {
+			return
+		}
+
+		await this.loadEnvelopes()
+		logger.debug(`syncing mailbox ${this.mailbox.databaseId} (${this.searchQuery}) after mount`)
+		await this.sync(false)
+		this.$store.commit('setHasFetchedInitialEnvelopes', true)
 	},
 	destroyed() {
 		this.bus.off('load-more', this.onScroll)