Skip to content
This repository has been archived by the owner on Apr 30, 2018. It is now read-only.

Emoticons Replace Only Works for the First One (Minor Bug) #20

Open
eniebla opened this issue Dec 4, 2015 · 2 comments
Open

Emoticons Replace Only Works for the First One (Minor Bug) #20

eniebla opened this issue Dec 4, 2015 · 2 comments

Comments

@eniebla
Copy link

eniebla commented Dec 4, 2015

If you send :) :) :) only the first one is replaced for an emoticon

FILE: jquery.chatjs.messageboard.js
var emoticons = [
{ pattern: ":-)", cssClass: "happy" },

MUST BE
{ pattern: /:-)/g, cssClass: "happy" },

replace all the "..." for /...../g

Hope this helps

@eniebla eniebla changed the title Emoticons Replace Only Works for de First One (Minor Bug) Emoticons Replace Only Works for the First One (Minor Bug) Dec 4, 2015
@rpaschoal
Copy link
Contributor

@eniebla, thank you! I did not notice this happening before!

I guess this project is getting very visible and we should put more effort on it to take out all those minor bugs...

@Andrewer11
Copy link

Working emoticons:
:)
:(
:D
:|
I thinks thats all

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants