Skip to content

Commit 096c97f

Browse files
authored
Merge pull request #11356 from nextcloud/fix/thread-heading
fix: thread heading alignment and participants
2 parents 70a04cd + 989f540 commit 096c97f

File tree

4 files changed

+14
-9
lines changed

4 files changed

+14
-9
lines changed

src/components/MailboxThread.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@
109109
</div>
110110
</template>
111111

112-
<Thread v-if="showThread" @delete="deleteMessage" />
112+
<Thread v-if="showThread" :current-account-email="account.emailAddress" @delete="deleteMessage" />
113113
<NoMessageSelected v-else-if="hasEnvelopes" />
114114
</AppContent>
115115
</template>

src/components/Message.vue

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -180,7 +180,7 @@ export default {
180180
}
181181
182182
.reply-buttons {
183-
margin: 26px 30px 0 50px;
183+
margin: 0 calc(var(--default-grid-baseline) * 4) calc(var(--default-grid-baseline) * 2) calc(var(--default-grid-baseline) * 14);
184184
display: flex;
185185
flex-wrap: wrap;
186186
gap: 5px;
@@ -205,7 +205,6 @@ export default {
205205
206206
&__notsuggested {
207207
margin-inline-start: auto;
208-
margin-inline-end: -12px;
209208
}
210209
}
211210
@media screen and (max-width: 600px) {

src/components/MessageHTMLBody.vue

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -173,8 +173,7 @@ export default {
173173
<style lang="scss" scoped>
174174
// account for 8px margin on iframe body
175175
.html-message-body {
176-
margin-inline-start: 50px;
177-
margin-top: 2px;
176+
margin : 2px calc(var(--default-grid-baseline) * 2) 0 calc(var(--default-grid-baseline) * 14);
178177
background-color: #FFFFFF;
179178
}
180179

src/components/Thread.vue

Lines changed: 11 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -93,6 +93,12 @@ export default {
9393
ThreadEnvelope,
9494
Popover,
9595
},
96+
props: {
97+
currentAccountEmail: {
98+
type: String,
99+
required: true,
100+
},
101+
},
96102
97103
data() {
98104
return {
@@ -165,7 +171,7 @@ export default {
165171
threadParticipants() {
166172
const recipients = this.thread.flatMap(envelope => {
167173
return envelope.from.concat(envelope.to).concat(envelope.cc)
168-
})
174+
}).filter(participant => participant.email !== this.currentAccountEmail)
169175
return uniqBy(prop('email'), recipients)
170176
},
171177
threadSubject() {
@@ -558,7 +564,7 @@ export default {
558564
559565
.mail-message-body {
560566
flex: 1;
561-
margin-bottom: calc(var(--default-grid-baseline) * 7);
567+
margin-bottom: calc(var(--default-grid-baseline) * 2);
562568
position: relative;
563569
}
564570
@@ -599,7 +605,8 @@ export default {
599605
// initial width
600606
width: 0;
601607
// while scrolling, the back button overlaps with subject on small screen
602-
padding-inline-start: calc(var(--border-radius-container-large) + var(--header-height));
608+
// 66px to allign with the sender Envelope -> 8px margin + 2px border+ avatar -> 40px width + envelope__header -> 8px padding + sender-> margin 8px
609+
padding-inline-start: 66px;
603610
// grow and try to fill 100%
604611
flex: 1 1 auto;
605612
h2,
@@ -639,7 +646,7 @@ export default {
639646
}
640647
641648
#mail-content {
642-
margin: calc(var(--default-grid-baseline) * 2) calc(var(--default-grid-baseline) * 10) 0 calc(var(--default-grid-baseline) * 15);
649+
margin: calc(var(--default-grid-baseline) * 2) calc(var(--default-grid-baseline) * 10) 0 calc(var(--default-grid-baseline) * 14);
643650
}
644651
645652
#mail-content iframe {

0 commit comments

Comments
 (0)