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

Some bug fixes, add keyboard control to mention_menu, and some css changes #240

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mikyxdxd
Copy link

@mikyxdxd mikyxdxd commented Jun 11, 2016

-bug fix

  1. window.onload 的 trigger 时间 大约要~10s 所以 MentionMenu 通常要等待10s后才会被append到body上 可能是我人在国外的关系 所以有些css js load得比较慢 希望国内的同学帮忙测试一下 我把window.onload删除了。 现在 append 之前只判断 window.$ != null
  2. mention_menu 会一直存在 就算你切换了联系人 我加了一个 hideMenu 的 function 挂在了 editAreaBlur 上, 只要editArea 失去 focus 那么mention_menu 就会hide()

-新加功能

现在mention_menu 能通过 ArrowUp
ArrowDown Space 和 Escape 来控制 空格可以直接选中hovered项 Escape 用来hide menu

-另外

  1. 修改了一些 mention_menu 的 css
  2. 添加了一个 .hover 为了实现按键控制menu
  3. 把emoji 的size 改大了一些 以前是 width:60px 现在改成了 120px 看起来好看一些

-Wishlist
如果menu_list太长 你在用ArrowDown 或者 ArrowUp 它不会自动Scroll 我有时间把这个加上去

希望有同学能帮忙测试一下

@geeeeeeeeek
Copy link
Owner

赞,看上去不错,我有空验证一下

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

Successfully merging this pull request may close these issues.

2 participants