Emoji picker #259
Emoji picker #259
Conversation
src/utils/contentEditable.ts
Outdated
| end: start + range.toString().length | ||
| } | ||
| } | ||
| export const isFirefox = () => navigator.userAgent.search("Firefox") > 0; |
There was a problem hiding this comment.
| export const isFirefox = () => navigator.userAgent.search("Firefox") > 0; | |
| export const isFirefox = () => navigator.userAgent.search("Firefox") > 0; |
| } | ||
|
|
||
| return ( | ||
| <div className="rcw-sender"> | ||
| <div ref={refContainer} className="rcw-sender"> | ||
| <button className='rcw-picker-btn' type="submit" onClick={handlerPressEmoji}> |
There was a problem hiding this comment.
I think we should add a conditional to hide the emoji picker setting it to true by default. What do you think?
| el.innerHTML = '' | ||
| setEnter(false) |
There was a problem hiding this comment.
| el.innerHTML = '' | |
| setEnter(false) | |
| el.innerHTML = ''; | |
| setEnter(false); |
| setEnter(false) | ||
| } | ||
| else if(brRegex.test(el.innerHTML)){ | ||
| el.innerHTML = el.innerHTML.replace(brRegex, '') |
There was a problem hiding this comment.
| el.innerHTML = el.innerHTML.replace(brRegex, '') | |
| el.innerHTML = el.innerHTML.replace(brRegex, ''); |
| el.innerHTML = el.innerHTML.replace(brRegex, '') | ||
| } | ||
| } | ||
| checkSize() |
There was a problem hiding this comment.
| checkSize() | |
| checkSize(); |
| > p { | ||
| white-space: pre-wrap; | ||
| } |
There was a problem hiding this comment.
Is it ok to delete this? Won't be a problem when using markdown?
There was a problem hiding this comment.
Yes its ok, this style I added in before PR.
src/utils/contentEditable.ts
Outdated
| const position = getCaretIndex(el) | ||
| let characterToEnter = '\n\n'; | ||
| let prevChar, char = ''; | ||
| if (position > 0) { // Change this |
|
Hi, awesome job with react-chat-widget. Is a question about emoji feature, can we set their properties like is designed on emoji-mart git page? If yes, how can i do that? Would be nice to set i18n propertie to do some translation on it. Thanks note: maybe i should have posted this in the questions tab, sorry |
Summary
Screenshots
Chrome
Firefox
Safari