Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

other way? #74

Open
wants to merge 64 commits into
base: content-editable
Choose a base branch
from
Open

other way? #74

wants to merge 64 commits into from

Conversation

ellscha
Copy link

@ellscha ellscha commented Dec 3, 2018

No description provided.

Method-es and others added 30 commits May 25, 2015 13:24
Reduced use of wrapper when not needed
Added resize event to reposition
Positioning, Reduced wrapper, Updated demo
triggers keyup event after emoji input for event binding
For/in, at least in our environment, winds up iterating over the
object's prototype methods such as `_super` and `nextObject`. These
result in a _bunch_ of `class="emoji-undefined"` elements being added to
the UI which makes the sections scroll way farther than needed.

Switching for/in to a normal for loop because these are just arrays so
why bother with for/in and hasOwnProperty?
Use normal for loop instead of for/in
jenwilhelm and others added 30 commits January 7, 2016 17:59
Return better name of plugin instance. 🐸
Adds onShow/onHide handlers 🍹
Destroy an emoji picker instance.
Fire an actual change event on emoji input
If you clicked on the grey background color area of a particular emoji and not the emoji itself, the wrong emoji would be spit out on the screen. Code I added makes sure that the right emoji is written out.
Checks for mobile before calling destroy()
Fix a bug related to clicking on emojis
adding in the picker functionality like the click and the search
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants