#2903 - Change detection refresh improvement

Identifier	#2903
Issue type	Trivial issue (does not break functionality)
Title	Change detection refresh improvement
Status	Completed
Tags	Roadmap: v11 (custom) Type: Usability (major) (custom)
Handling member	Salman
Addon	core
Description	The current change detection method utilizes iframes and results in the refresh of the comment / quick reply boxes... hence you added a block for when such elements were activated by the user. The problem with this is obvious. If someone is typing a reply, they'll no longer be able to see if someone makes a post. Thus, I have two possible solutions for this: a) Prevent quick reply / comment box from being a part of the iframe that gets refreshed by change detection, and don't stop the refresh when those elements are activated. or b) Instead of using iframe refreshes, show updated content via. AJAX (example... activity feed).
Steps to reproduce
Funded?	No

The system will post a comment when this issue is modified (e.g., status changes). To be notified of this, click "Enable comment notifications".

Unrated

Comments

By Guest posted 27th Oct 2016, 5:33 PM

Drawback with using AJAX method: It becomes complicated if we use change_detection to detect deletions or edits.

By Guest posted 27th Oct 2016, 5:49 PM

This could be a personal error on my side. I might have included quick reply / comment box by accident into the internalise_ajax_screen function.

By Guest posted 28th Oct 2016, 10:57 AM

Not an error of yours, it does work via including the whole Composr page into the iframe.

I agree with 'b'.

By Guest posted 14th Mar 2021, 8:25 PM

This was implemented by Salman way back. I've just reviewed, it looks good.