Nested inline code, emoji and mentions disappear inside bold, italic and strikethrough
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 75/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- javascript
- Ambito
- frontend
Direzione di ricerca
Inizia nel pacchetto markups in BoldSpan.js, ItalicSpan.js e StrikeSpan.js; leggi come ciascuno mappa i tipi di figli AST e dove restituisce null per i figli non gestiti. Riproduci con l’output del parser per *run yarn build first*, poi aggiungi i casi mancanti INLINE_CODE, EMOJI e MENTION, più i timestamp in StrikeSpan.js, con test di regressione. Fatto significa che il contenuto annidato viene renderizzato e che i test di markup esistenti passano; i link dentro italic/strikethrough e i crash di nested-emphasis sono fuori ambito.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Description
Nested inline code, emoji and mentions are silently dropped by the markup renderer inside bold, italic and strikethrough.
For example, this message:
*run `yarn build` first*
renders as "run first", with the inline code missing.
Steps to reproduce
- Use EmbeddedChat develop at commit 1cb7bb38.
- Parse the example above using @rocket.chat/message-parser 0.31.29.
- Render the resulting AST through the markups package.
- Observe that the parser includes an INLINE_CODE child inside BOLD, but the rendered output drops it.
I also reproduced missing inline code, emoji and mentions inside italic and strikethrough using a local rendering harness.
Expected behaviour
Supported nested inline content should remain visible inside the formatting wrapper.
Environment
- EmbeddedChat: develop at 1cb7bb38
- Node.js: 24.0.0
- Yarn: 3.6.4
- @rocket.chat/message-parser: 0.31.29
Additional context
BoldSpan.js, ItalicSpan.js and StrikeSpan.js return null for child types they do not explicitly handle.
I have a local fix with regression tests for inline code, emoji and mentions inside these wrappers, plus timestamps inside strikethrough. Links inside italic/strikethrough and the pre-existing CommonJS nested-emphasis crashes remain outside this fix.
Related PRs #1044 and #1139 overlap with some files but address different behaviour.
Verification so far uses local package rendering, not a live Rocket.Chat server or the real chat UI.
Is this a fix you'd welcome a PR for?
- Lingua principale
- JavaScript
- Stelle
- 170
- Fork
- 397
- Merge medio
- 12g 14h
- PR unite (30g)
- 6
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Nessuna guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di RocketChat/EmbeddedChat
-
docs: README says Node 22 is required, but .nvmrc pins v24 and preinstall enforces itForse già presa @Veerpratapgupta l’ha presa 29 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
RocketChat/EmbeddedChat#1377 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
test: isMessageSequential in layout_editor has no coverageForse di nuovo libera Una pull request per questa issue è stata chiusa senza essere unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
RocketChat/EmbeddedChat#1373 ·
I maintainer di solito rispondono entro 1 giorno
-
Permissions change-detection in useFetchChatData is dead — applyPermissions re-runs on every callForse già presa @yash113gadia l’ha presa 166 giorni fa. Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
RocketChat/EmbeddedChat#1317 ·
I maintainer di solito rispondono entro 1 giorno
-
Only the last emoji shortname in a message is converted to unicodeForse già presa @yash113gadia l’ha presa 166 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
RocketChat/EmbeddedChat#1315 ·
I maintainer di solito rispondono entro 1 giorno
-
bug: "Last login" shows ambiguous weekday with no date for logins older than todayForse già presa @yash113gadia l’ha presa 167 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
RocketChat/EmbeddedChat#1313 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di RocketChat/EmbeddedChat
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
-
Bug: Table layout and date column are cut off on mobile screens on the Tournaments -> History pageApertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
hexlet-codebattle/codebattle#2375 ·
-
intake mcp-intake needs-human-review priority:high
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
Ikalus1988/MisakaNet#5041 · 3 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
nicobailon/pi-web-access#682 ·
I maintainer di solito rispondono entro 1 giorno
-
nip27: long hashtag swallows the rest of the content, lone # gives an empty hashtagForse già presa @bardrop l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
nbd-wtf/nostr-tools#578 ·