View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0002903 | Composr | core | public | 2016-10-27 21:26 | 2021-03-15 00:25 |
| Reporter | Patrick Schmalstig | Assigned To | Salman | ||
| Severity | @40@ | ||||
| Status | resolved | Resolution | fixed | ||
| Product Version | |||||
| Fixed in Version | |||||
| Summary | 0002903: Change detection refresh improvement | ||||
| 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). | ||||
| Tags | Roadmap: v11, Type: Usability (major) | ||||
| Time estimation (hours) | 16 | ||||
| Sponsorship open | |||||
| related to | 0002904 | non-assigned | Change detection for forum posts | |
| related to | 0002905 | resolved | Chris Graham | Show message on change detection refresh |
| related to | 0002906 | closed | Chris Graham | No automatic refresh on change detection; let user decide |
|
|
Drawback with using AJAX method: It becomes complicated if we use change_detection to detect deletions or edits. |
|
|
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. |
|
|
Not an error of yours, it does work via including the whole Composr page into the iframe. I agree with 'b'. |
|
|
This was implemented by Salman way back. I've just reviewed, it looks good. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2016-10-27 21:26 | Patrick Schmalstig | New Issue | |
| 2016-10-27 21:30 | Patrick Schmalstig | Relationship added | related to 0002904 |
| 2016-10-27 21:32 | Patrick Schmalstig | Relationship added | related to 0002905 |
| 2016-10-27 21:33 | Patrick Schmalstig | Note Added: 0004489 | |
| 2016-10-27 21:38 | Patrick Schmalstig | Relationship added | related to 0002906 |
| 2016-10-27 21:49 | Patrick Schmalstig | Note Added: 0004491 | |
| 2016-10-28 14:57 | Chris Graham | Note Added: 0004496 | |
| 2016-10-28 14:57 | Chris Graham | Summary | change detection refresh improvement => Change detection refresh improvement |
| 2016-10-28 14:57 | Chris Graham | Time estimation (hours) | => 16 |
| 2017-05-01 16:10 | Chris Graham | Tag Attached: Type: Usability (major) | |
| 2019-11-15 20:50 | Chris Graham | Tag Attached: Roadmap: v11 | |
| 2019-11-15 20:50 | Chris Graham | Assigned To | => Salman |
| 2019-11-15 20:50 | Chris Graham | Status | non-assigned => assigned |
| 2021-03-15 00:25 | Chris Graham | Status | assigned => resolved |
| 2021-03-15 00:25 | Chris Graham | Resolution | open => fixed |
| 2021-03-15 00:25 | Chris Graham | Note Added: 0006995 |
