QuoteME ! 2.1.1

[cclaerhout]

Chrome 34+35 OSX and Win 8.
@rellect does it happen with FF as well?

I've tried with Chrome 35 (win 7 - shouldn't make any difference), with the direct mode and the "after click" mode, it's working.
Test protocole:
Use an existed thread, quote the text (XenForo default function or QuoteMe), press the button "post reply", quote again some text (XenForo default function or QuoteMe) => working for me (at least with mce4).

 

[cclaerhout]

Chrome 34+35 OSX and Win 8.
@rellect does it happen with FF as well?

It seems to be a problem with the Redactor main JS file. I will check if I can do something.

Edit:
The original Redactor insertHtml command doesn't work when the editor is not active (fade in/fade out) and triggers an error because it can't find the nodetype of the current caret position since it doesn't exist yet.

 

[refael]

Chrome 34+35 OSX and Win 8.
@rellect does it happen with FF as well?

Can't reproduce the issue in firefox, seems to be chrome specific.

 

[cclaerhout]

@Moshe1010
Github commit
To download: https://github.com/cclaerhout/xen_QuoteME (bottom right).
This workaround should fix the problem.

 

[Moshe1010]

Thanks, I will check it out

 

[cclaerhout]

Thanks, I will check it out

Thanks, I'm waiting for your feedback.

 

[Moshe1010]

Thanks, I'm waiting for your feedback.

Clicking 3 times with the mouse makes the entire page selected not just the text. I will send you a PM

 

[cclaerhout]

Clicking 3 times with the mouse makes the entire page selected not just the text. I will send you a PM

I can't reproduce this and the modification done in the Github release could not have done that.
Please also confirm that the original problem you reported is fixed or not. Thanks.

 

[Moshe1010]

I can't reproduce this and the modification done in the Github release could not have done that.
Please also confirm that the original problem you reported is fixed or not. Thanks.

The original problem resolved. Have you tried triple click with my issue?

Edit: K, never mind, I figured this out. All good now, thanks.

 

[Newt]

That's neat! At first I thought it would let post the quoted text on social media though

 

[Floyd R Turbo]

Installed, working well, great add-on!

I did have a question though, in the d/l package there is the upload dir and then another folder called 'extras' and appears to be related to this but is this something that needs to go somewhere and be installed, or is that related to another add-on?

Also the readme.txt file is way out of date, it doesn't explain these things, and it still references the need to install TMS which is only required for xF 1.1

Functionally, love it!

 

[JJJ]

Somehow, I no longer can get the direct quoting mode to work.

Whatever I set for QuoteME Activation... I always have to click that "Fast Quotes", to get it working

 

[moleculo]

Somehow, I no longer can get the direct quoting mode to work.

Whatever I set for QuoteME Activation... I always have to click that "Fast Quotes", to get it working

Same here. It doesn't seem to be working anymore either - not sure if it's a browser issue? I'm testing on Chrome right now.

 

[cclaerhout]

Tested - can't reproduce the issue. Check your browser console to see if there's any JS error that would prevent the script to load.

 

[Moshe1010]

Works fine with Chrome Version 36.0.1985.125 and XF 1.3.4

 

[HWS]

We also do not see any errors with this add-on.

 

[owtlawatv]

thanks for this addon....

 

[Janmaat]

There is also an issue on mobile devices. Found out with iOS 7.

The quoted text snippet will be copied into the editor field correctly. But the quoted user id and the quoted message id will point to a different user and post, not to the user/post that was originally quoted.

 

[cclaerhout]

There is also an issue on mobile devices. Found out with iOS 7.

The quoted text snippet will be copied into the editor field correctly. But the quoted user id and the quoted message id will point to a different user and post, not to the user/post that was originally quoted.

Confirmed, thanks.

 

[cclaerhout]

@Janmaat
Fixed, but I need a few time to repack the js.