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

To keep the selected li visible under the dropdown, using keyboard up/down arrow keys when a large number of options are available. #83

Open
wants to merge 81 commits into
base: 3.5
Choose a base branch
from

Conversation

shivarchit
Copy link

Hi, I'd a requirement which wanted me to keep the selected option always visible in the dropdown when using the keyboard up/down keys. I have added a change which now allows me to change the scroll and keep the selected option visible.

StevenDevooght and others added 30 commits August 7, 2013 20:31
in ie9 and probably (ie10) when you click on an option in the pulldown menu the focus of the editor is lost and the selected option can not be added to the tinymce editor. This single line fixes this issue.
added focus in select function for IE
StevenDevooght and others added 30 commits July 12, 2016 18:59
Insert function can use a different value than the queryBy field. (insertFrom)
Define sorter as option to override the default 'sorter' function
Use HTTPS connection for loading spinner to prevent mixed content
It can be useful when we want to set a different HTML according to the item position.

Here is an example to active the first item:

render: function(index, item) {
  return '<li' + (index == 0 ? ' class="active"' : '') + '>' +
           '<a href="javascript:;"><span>' + item[this.options.queryBy] + '</span></a>' +
         '</li>';
}

Signed-off-by: Nicolas Dos Santos <[email protected]>
…ine is impossible when there is already text on this line

In the prevCharIsSpace function, 'start' variable can be lower than 0, so it computes a substring with a negative value as first argument
update the render function to add the item index in argument.
Adding a new mention at the beginning of a line is impossible when there is already text on this line
Fixed bug where last highlighted item was being added
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.

6 participants