View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0002902 | Composr | core | public | 2016-10-26 01:58 | 2016-10-28 16:47 |
Reporter | Patrick Schmalstig | Assigned To | Chris Graham | ||
Severity | Minor-bug | ||||
Status | closed | Resolution | unable to reproduce | ||
Product Version | |||||
Fixed in Version | |||||
Summary | 0002902: If change detection returns 1, but refresh not to happen, notification sound plays repeatedly | ||||
Description | If change detection script detects a change on the page you're on (eg. tickets, or in my case the forums), and due to some circumstance, the content is not to be refreshed (such as activating quick reply), the new notification sound keeps playing every time change detection script runs. | ||||
Additional Information | Chrome Windows 10 lovinity.org Composr CMS 10 RC20 | ||||
Tags | No tags attached. | ||||
Time estimation (hours) | |||||
Sponsorship open | |||||
related to | 0002905 | resolved | Chris Graham | Show message on change detection refresh |
|
I can't reproduce or see an issue in the code. INTERNALISED_AJAX_SCREEN.tpl has changed now. Maybe you need to update your code to reflect that? INTERNALISED_AJAX_SCREEN.tpl setup interval calling detect_change in internalised_ajax_screen.js internalised_ajax_screen.js detects change clears interval calls callback in INTERNALISED_AJAX_SCREEN.tpl INTERNALISED_AJAX_SCREEN.tpl if active quick reply post, stops calls block refresh calls _detected_change in internalised_ajax_screen.js internalised_ajax_screen.js makes sound As you can see, the interval gets cleared (so no loop), and the sound only ever happens if there's no active quick reply (so no sound). |
Date Modified | Username | Field | Change |
---|---|---|---|
2016-10-26 01:58 | Patrick Schmalstig | New Issue | |
2016-10-26 01:59 | Patrick Schmalstig | Additional Information Updated | View Revisions |
2016-10-27 21:41 | Patrick Schmalstig | Relationship added | related to 0002905 |
2016-10-28 16:47 | Chris Graham | Note Added: 0004498 | |
2016-10-28 16:47 | Chris Graham | Status | non-assigned => closed |
2016-10-28 16:47 | Chris Graham | Assigned To | => Chris Graham |
2016-10-28 16:47 | Chris Graham | Resolution | open => unable to reproduce |