Skip to content

Commit 8553a13

Browse files
authored
Merge pull request #6962 from nextcloud-libraries/chore/remove-deprecated-prop-ncrich
chore!(NcRichContenteditable): remove `title` prop in favor of `label`
2 parents 6f238b0 + 055d1a0 commit 8553a13

File tree

3 files changed

+7
-28
lines changed

3 files changed

+7
-28
lines changed

CHANGELOG.md

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,7 @@ The `richEditing` mixin can be replaced by just using the `NcRichText` component
138138
- The `limitWidth` was removed (the content is now always limited width) [\#5605](https://github.com/nextcloud-libraries/nextcloud-vue/pull/5605)
139139
- The `isFullscreen`, and `isMobile` mixins were removed. Use the according composables instead.
140140
- The `clickOutsideOptions` mixin is removed
141-
- The `box-sizing: border-box` is now default for following components and its content. This is done to match behaviour of `NcContent` (as they can be mounted directly to `body`):
141+
- The `box-sizing: border-box` is now default for following components and its content. This is done to match behavior of `NcContent` (as they can be mounted directly to `body`):
142142
- `NcModal`
143143
- `NcPopover`
144144
- `NcDateTimePicker`
@@ -148,6 +148,9 @@ The `richEditing` mixin can be replaced by just using the `NcRichText` component
148148
- `NcPopover` is no longer a transparent wrapper over the `floating-vue` package.
149149
Instead only use the documented properties and events.
150150
If you find some use cases not covered by the documented interface, please open a feature request.
151+
- `NcRichContenteditable`
152+
- `NcAutoCompleteResult`: The `title` prop was deprecated and is now removed in favor of the `label` prop
153+
- `NcMentionBubble`: The `title` prop was deprecated and is now removed in favor of the `label` prop
151154
- `NcSelect`
152155
- `userSelect` property was removed, instead just use the `NcSelectUsers` component
153156
- `closeOnSelect` property was removed in favor of `keepOpen`.

src/components/NcRichContenteditable/NcAutoCompleteResult.vue

Lines changed: 2 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,8 @@
2020

2121
<!-- Label and subline -->
2222
<span class="autocomplete-result__content">
23-
<span class="autocomplete-result__title" :title="labelWithFallback">
24-
{{ labelWithFallback }}
23+
<span class="autocomplete-result__title" :title="label">
24+
{{ label }}
2525
</span>
2626
<span v-if="subline" class="autocomplete-result__subline">
2727
{{ subline }}
@@ -43,14 +43,6 @@ export default {
4343
},
4444
4545
props: {
46-
/**
47-
* @deprecated Use `label` instead
48-
*/
49-
title: {
50-
type: String,
51-
required: false,
52-
default: null,
53-
},
5446
label: {
5547
type: String,
5648
required: false,
@@ -91,10 +83,6 @@ export default {
9183
? this.getAvatarUrl(this.id, 44)
9284
: null
9385
},
94-
// For backwards compatibility
95-
labelWithFallback() {
96-
return this.label || this.title
97-
},
9886
},
9987
10088
methods: {

src/components/NcRichContenteditable/NcMentionBubble.vue

Lines changed: 1 addition & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
class="mention-bubble__icon" />
1616

1717
<!-- Title -->
18-
<span role="heading" class="mention-bubble__title" :title="labelWithFallback" />
18+
<span role="heading" class="mention-bubble__title" :title="label" />
1919
</span>
2020

2121
<!-- Selectable text for copy/paste -->
@@ -35,14 +35,6 @@ export default {
3535
type: String,
3636
required: true,
3737
},
38-
/**
39-
* @deprecated Use `label` instead
40-
*/
41-
title: {
42-
type: String,
43-
required: false,
44-
default: null,
45-
},
4638
label: {
4739
type: String,
4840
required: false,
@@ -80,10 +72,6 @@ export default {
8072
? `@${this.id}`
8173
: `@"${this.id}"`
8274
},
83-
// Fallback to title for compatibility
84-
labelWithFallback() {
85-
return this.label || this.title
86-
},
8775
},
8876
8977
methods: {

0 commit comments

Comments
 (0)