Skip to content
This repository was archived by the owner on Sep 11, 2024. It is now read-only.

Commit 6ac366e

Browse files
authoredApr 3, 2023
Style mentions as pills in rich text editor (#10448)
1 parent f933806 commit 6ac366e

File tree

1 file changed

+30
-2
lines changed

1 file changed

+30
-2
lines changed
 

Diff for: ‎res/css/views/rooms/wysiwyg_composer/components/_Editor.pcss

+30-2
Original file line numberDiff line numberDiff line change
@@ -101,9 +101,37 @@ limitations under the License.
101101
}
102102
}
103103

104-
/* this selector represents what will become a pill */
104+
/* this selector represents what will become a pill
105+
nb despite there being mx_UserPill and mx_RoomPill classes appended to these pills
106+
in the current composer, there don't appear to be any styles associated with those classes
107+
in this repo */
105108
a[data-mention-type] {
106-
cursor: text;
109+
/* these entries duplicate mx_Pill from _Pill.pcss */
110+
padding: $font-1px 0.4em;
111+
line-height: $font-17px;
112+
border-radius: $font-16px;
113+
vertical-align: text-top;
114+
/* TODO turning this on hides the cursor from the composer for some
115+
reason, so comment out for now and assess if it's needed when we add
116+
the Avatars
117+
display: inline-flex;
118+
align-items: center; not required with the above turned off
119+
120+
Potential fix is using display: inline, width: fit-content
121+
*/
122+
box-sizing: border-box;
123+
max-width: 100%;
124+
overflow: hidden;
125+
126+
color: $accent-fg-color;
127+
background-color: $pill-bg-color;
128+
129+
/* combining the overrides from _BasicMessageComposer.pcss */
130+
user-select: all;
131+
position: relative;
132+
cursor: unset; /* We don't want indicate clickability */
133+
text-overflow: ellipsis;
134+
white-space: nowrap;
107135
}
108136
}
109137

0 commit comments

Comments
 (0)
This repository has been archived.