yabwe / medium-editor

Medium.com WYSIWYG editor clone. Uses contenteditable API to implement a rich text solution.
https://yabwe.github.io/medium-editor/
Other
16.04k stars 1.85k forks source link

Toolbar doesn't disappear immediately after deselecting text #287

Closed hql287 closed 9 years ago

hql287 commented 10 years ago

I tried to integrate this with one of my rails projects and this happens: untitled mov

Basically, when I select a piece of text, the toolbar immediately shows up as expected but when I click somewhere else (to deselect the text) it takes about ~500ms for the toolbar to disappear. Any idea why this happens?

I tried using the gem as well but got the same result. Also, it works fine on normal project and I only encounter this when trying to make it work in a rails project. Could this be some kind of conflict with other JS libraries?

Thanks!

hql287 commented 10 years ago

Here is the list of JS libs that I use

//= require i18n //= require i18n/translations //= require jquery //= require jquery_ujs //= require js-routes //= require bootstrap/dropdown //= require bootstrap/modal //= require bootstrap/tooltip //= require bootstrap/popover //= require birthdaypicker/bday-picker //= require lodash/dist/lodash //= require jquery-file-upload/js/vendor/jquery.ui.widget //= require jquery-file-upload/js/jquery.fileupload //= require jquery-file-upload/js/jquery.iframe-transport //= require jquery-file-upload/js/jquery.fileupload-process //= require jquery-file-upload/js/jquery.fileupload-validate //= require jcrop/js/jquery.Jcrop //= require imagesloaded/imagesloaded.pkgd //= require jquery-masonry/dist/masonry.pkgd //= require parsley //= require parsley.remote //= require holderjs/holder //= require drop/drop //= require bootbox/bootbox //= require selectize/dist/js/standalone/selectize //= require galleria/src/galleria //= require elevatezoom/jquery.elevatezoom.js //= require jquery.idTabs.min //= require datatables/media/js/jquery.dataTables.js //= require moment/moment.js //= require iOS-Overlay/js/iosOverlay.js //= require spin.js/spin.js //= require list.js/dist/list.js //= require list.fuzzysearch.js/dist/list.fuzzysearch.js //= require jquery-tags-input/jquery.tagsinput.js //= require jquery-timepicker-jt/jquery.timepicker.js //= require blueimp-load-image/js/load-image.min.js //= require awesome-cropper/build/jquery.awesome-cropper.js //= require awesome-cropper/components/imgareaselect/scripts/jquery.imgareaselect.js //= require bootstrap-tagsinput/dist/bootstrap-tagsinput.js //= require pnotify/pnotify.core.js //= require bootstrapValidator/dist/js/bootstrapValidator.min.js //= require bootstrapValidator/dist/js/language/vi_VN.js //= require jquery-autosize/jquery.autosize.js //= require medium-editor/dist/js/medium-editor.js //= require typeahead //= require provinces //= require util //= require user //= require conversation //= require productList //= require_tree .

nmielnik commented 9 years ago

@hungle88 This hopefully should have improved after #361 and #365. Would you be able to verify and potentially close this issue?

There will be some lag, but it should hopefully be much less noticeable (only 50ms)

hql287 commented 9 years ago

Hi @nmielnik

Thanks for getting back to me. However, due to our timeline, we had to choose another solution for this :-( Will save this for our next project, though.

Cheers!