react-native-zss-rich-text-editor icon indicating copy to clipboard operation
react-native-zss-rich-text-editor copied to clipboard

Running slowly on Android

Open ColdStar007 opened this issue 5 years ago • 20 comments

It works fine on iOS, but it runs very slowly on Android.

ColdStar007 avatar Dec 04 '18 03:12 ColdStar007

same issue

minhphung210 avatar Dec 07 '18 02:12 minhphung210

Same here !

antony-meunier avatar Dec 07 '18 13:12 antony-meunier

@ColdStar007 @minhphung210 @antony-meunier There is an infinite loop running in the android version, it has been resolved in a few different forks but there is little to no activity in this repo unfortunately.

You can depend on the tag 1.1.1 in my fork of this repo which has a fix.

https://github.com/arneson/react-native-zss-rich-text-editor.git#1.1.1

arneson avatar Dec 10 '18 12:12 arneson

@arneson arn your solution not work for me :( thank for reply

minhphung210 avatar Dec 12 '18 03:12 minhphung210

That's unfortunate. Have you made sure to do a clean native build before testing? As the problem is in the editor.html file in the library which is only copied over on build, not reload.

arneson avatar Dec 12 '18 06:12 arneson

@arneson keen to make a hard fork? I'm having issues. lets collaborate I have a fork too: https://github.com/export-mike/react-native-zss-rich-text-editor

I'm keen to change this to use the recommended react-native-web-view which is faster for ios and also has window.postMessage working without hacks.

export-mike avatar Dec 14 '18 13:12 export-mike

@arneson where is the change to remove the infinite loop? https://github.com/arneson/react-native-zss-rich-text-editor/commits/master/src/editor.html

export-mike avatar Dec 14 '18 13:12 export-mike

@export-mike

Yeah, I'd love to get this library working properly again. Would love to collaborate.

It's the commit with message fix getCaretPosition.

https://github.com/arneson/react-native-zss-rich-text-editor/commit/7704a0efdce8f0a41f26526c5a4558a6a69c735b#diff-5764eca5893a91bd5fb61afbae89595d

Worth noting is that I did not write that commit but pulled it from charpeni 's fork.

arneson avatar Dec 14 '18 13:12 arneson

ok cool sounds great.

basically the fork of the editor I'm using has the ability to render custom buttons. I need to change it to use the react-native-web-view

I can pull in this infinite loop fix manually. what other changes have you introduced?

what functionality are you working to support/fix?

export-mike avatar Dec 14 '18 15:12 export-mike

Working on supporting the toolbar which behaves like a fucking tool in the current version.

But totally agree we need a new hard fork. I can create a fork and add you as a collab, that sound good?

14 dec. 2018 kl. 16:59 skrev Mike James [email protected]:

ok cool sounds great.

basically the fork of the editor I'm using has the ability to render custom buttons. I need to change it to use the react-native-web-view

I can pull in this infinite loop fix manually. what other changes have you introduced?

what functionality are you working to support/fix?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

arneson avatar Dec 14 '18 22:12 arneson

Yeah great. basically the work I have done so far allows custom components to be rendered.

  • Need to successfully remove the title editor from the editor if not required.
  • Show and hide toolbar successfully on android
  • Android Performance
  • iOS shows a black line page-break kinda thing when scrolling.

export-mike avatar Dec 14 '18 23:12 export-mike

Bottom 2 points need to be completed.

export-mike avatar Dec 14 '18 23:12 export-mike

@export-mike Don't think I've seen the last one. Could you send over a screen shot?

arneson avatar Dec 17 '18 09:12 arneson

@arneson https://join.slack.com/t/reactnativezs-ash8421/shared_invite/enQtNTA1NTMxNzE4NDY3LWM3MzQwM2ZkZTEwZjRhMGYyMDBmZTY2ZmMyYmY1ZGU0NGE4OGVjN2M4NTY1MDkyMzFmNzAxMDZlMmU2YmJlYTc

Let's move to slack

export-mike avatar Dec 17 '18 11:12 export-mike

Saw that link way too late. It’s dead now :) @export-mike

arneson avatar Dec 18 '18 08:12 arneson

@arneson https://join.slack.com/t/reactnativezs-ash8421/shared_invite/enQtNTA1NTMxNzE4NDY3LWM3MzQwM2ZkZTEwZjRhMGYyMDBmZTY2ZmMyYmY1ZGU0NGE4OGVjN2M4NTY1MDkyMzFmNzAxMDZlMmU2YmJlYTc

export-mike avatar Dec 18 '18 13:12 export-mike

@arneson Thanks your fork worked for me!

ethanyuwang avatar Jan 09 '19 23:01 ethanyuwang

@arneson your version supports hashtag and mentions?

ashokkumar88 avatar Jan 24 '19 12:01 ashokkumar88

@ashokkumar88 I have not added that functionality, so if it exists in original package then yes, otherwise no.

arneson avatar Jan 25 '19 11:01 arneson

looks like UIWebView has been deprecated. I'll be updating https://github.com/export-mike/react-native-zss-rich-text-editor to use WKWebView.

export-mike avatar Sep 01 '19 19:09 export-mike