[ZBX-8877] All filters hide/show do not work on IE11 Created: 2014 Oct 09 Updated: 2017 May 30 Resolved: 2014 Oct 16 |
|
Status: | Closed |
Project: | ZABBIX BUGS AND ISSUES |
Component/s: | Frontend (F) |
Affects Version/s: | 2.2.6 |
Fix Version/s: | 2.2.7rc1, 2.4.2rc1, 2.5.0 |
Type: | Incident report | Priority: | Major |
Reporter: | Alexey Pustovalov | Assignee: | Unassigned |
Resolution: | Fixed | Votes: | 0 |
Labels: | filters, js, regression | ||
Remaining Estimate: | Not Specified | ||
Time Spent: | Not Specified | ||
Original Estimate: | Not Specified | ||
Environment: |
Internet Explorer 11 |
Attachments: |
![]() ![]() |
||||
Issue Links: |
|
Description |
Since 2.2.6 all filters do not work on IE11. |
Comments |
Comment by Alexander Vladishev [ 2014 Oct 09 ] |
Works fine in latest 1.8, 2.0, 2.4 and trunk r.49720. |
Comment by Pavels Jelisejevs (Inactive) [ 2014 Oct 10 ] |
Couldn't reproduce the filter problem in the latest trunk. But, when you open a graph and press the left arrow on the timebar, the same error occurs. |
Comment by Krists Krigers (Inactive) [ 2014 Oct 15 ] |
Fixed in r49904, branch svn://svn.zabbix.com/branches/dev/ZBX-8877 (copy of branches/2.2). Should be tested in all available IEs as seemingly unnecessary function was removed. |
Comment by Ivo Kurzemnieks [ 2014 Oct 15 ] |
(1) String changes? kristsk No string changes. RESOLVED. iivs CLOSED. |
Comment by Ivo Kurzemnieks [ 2014 Oct 15 ] |
(2) browsers.js: Change back copyright address "021130-1301" to "02110-1301" kristsk RESOLVED in r49945. iivs CLOSED. |
Comment by Ivo Kurzemnieks [ 2014 Oct 15 ] |
(3) I couldn't find usage of "OP" (Opera), if we're not using it, we can remove it. Futher more it only detects old Opera versions 12.x not 24.x <richlv> keep in mind that opera 12.x is the latest on linux kristsk RESOLVED in r49946. iivs CLOSED. |
Comment by Ivo Kurzemnieks [ 2014 Oct 16 ] |
(4) Select something on a page or select all. Then click inside graph to select range. Notice how selection remains. Before all selection was deselected. See https://support.zabbix.com/secure/attachment/31010/deselectall.png At first it seemed to affect older IEs', but it affects other newer browsers and probably other front-end places aswell. Probably not a good change and a different solution is required. Code in gtlc.js: 1099-1107 is, however, unused. kristsk RESOLVED in r49962. iivs CLOSED. |
Comment by Ivo Kurzemnieks [ 2014 Oct 16 ] |
(5) regexp /trident.*rv:11\./ was also correct and probably is more correct than simply identifying browser version by MSHTML, although it changes every version. There is no guarantee that IE12 will not be the same trident/7.0 ,but use rv:12. Let's change it back. kristsk RESOLVED in r49962. iivs CLOSED. |
Comment by Ivo Kurzemnieks [ 2014 Oct 16 ] |
TESTED |
Comment by Krists Krigers (Inactive) [ 2014 Oct 16 ] |
Fixed and merged to:
|