Eric Han

Results 118 comments of Eric Han

## Proposal ### Please re-state the problem that we are trying to solve in this issue. Markdown heading is not allowed in quotes ### What is the root cause of...

Hi @Santhosh-Sellavel , would you like to have a look at my [proposal](https://github.com/Expensify/App/issues/17367#issuecomment-1514056856)? It identifies the root cause and provides a simple solution with minimal changes.

### Proposal ### Please re-state the problem that we are trying to solve in this issue. Editing comment `[block]` shows empty in the composer. ### What is the root cause...

Hi @rushatgabhane, I think `[block]` is the only special string that might conflict with user's input in the expensify-common lib.

@rushatgabhane One defect I can see about `` is that it'll be stripped by method Str.stripHTML before being processed in method replaceBlockWithNewLine, see [here](https://github.com/Expensify/expensify-common/blob/3cdaa947fe77016206c15e523017cd50678f2359/lib/ExpensiMark.js#L527-L531) ```js generatedMarkdown = generatedMarkdown.replace( /|||\n|||||||||||||||/gm, '[block]',...

Hi @rushatgabhane, would you like to have a look at my comment about the drawback of the `` solution here https://github.com/Expensify/App/issues/17776#issuecomment-1526606921? IMHO, I don’t think it’ll be working. Using ``...

@tienifr I think first you need to change the other basic method Str.stipHTML to get it work, which is not simple and is not a good pattern to fix an...

## Proposal ### Please re-state the problem that we are trying to solve in this issue. Comment disappears when edited in _* text _*. ### What is the root cause...

> > But after the backend corrects the overlapping html and compares with original html, it figures out that frontend sends the same html to save. So the backend returns...

> > > But I think the frontend fix is also necessary. > > Could you elaborate a bit more on this? If we change the back end to return...