Flicker-free screens
(ZBXNEXT-1327)
[ZBXNEXT-1253] Support of flicker-free displaying of screens in monitoring Created: 2012 Jun 05 Updated: 2017 May 31 Resolved: 2012 Oct 23 |
|
Status: | Closed |
Project: | ZABBIX FEATURE REQUESTS |
Component/s: | Frontend (F) |
Affects Version/s: | 2.1.0 |
Fix Version/s: | 2.0.3, 2.1.0 |
Type: | Change Request (Sub-task) | Priority: | Major |
Reporter: | Eduards Samersovs (Inactive) | Assignee: | Unassigned |
Resolution: | Fixed | Votes: | 3 |
Labels: | None | ||
Remaining Estimate: | Not Specified | ||
Time Spent: | Not Specified | ||
Original Estimate: | Not Specified |
Attachments: | time_period_incorrect_scrollbar.png | ||||||||||||||||||||||||
Issue Links: |
|
Description |
Flicker free means no full refresh of pages for screen displaying in monitoring. Only individual cells should be refreshed. Flicker free screen should deliver exactly the same look&feel as screens in Zabbix 2.0. Be careful when implementing support of URLs! All cells must be refreshed individually and absolutely instantly, no flickers are allowed. Even if we have 100x100 screen, there will be no flickering during refresh of individual cells. Time navigator will not cause page reload. New code base could be reused in the future to make screen elements embeddable into Dashboard. Logic and presentation must be separated for displaying cells of different types. Each cell-type must use separate file(s) for presentation. |
Comments |
Comment by richlv [ 2012 Jun 05 ] |
this is actually a dupe of |
Comment by Alexander Vladishev [ 2012 Jun 06 ] |
Comment by Eduards Samersovs (Inactive) [ 2012 Jun 19 ] |
Implemented in development branch svn://svn.zabbix.com/branches/dev/ZBXNEXT-1253 |
Comment by Eduards Samersovs (Inactive) [ 2012 Jun 19 ] |
(1) Select box changes not synchronized with scroll bar. <Eduard> RESOLVED <Vedmak> when some period selected in zoom, timeline can jump to incorrect time usually after currently selected period. <Eduard> RESOLVED + synchronized sbox formatting as in scrollbar. If you catch the bug with incorrect sbox time, please write use case how to reproduce this.. <Vedmak> CLOSED |
Comment by Alexei Vladishev [ 2012 Jun 19 ] |
(2) Undefined index: legend [include/classes/flickerfreescreens/CFlickerfreeScreenGraph.php:156] As seen in configuration of screens when selecting a simple pie graph. Graphs is broken and displays: Warning. Field "legend" is not integer. The URL from Apache log file chart6.php?graphid=503&width=400&height=100&legend=&graph3d=1&period=63072000. <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexei Vladishev [ 2012 Jun 19 ] |
(3) I have 2x2 screen displaying 4 simple graphs. I am unable to zoom-in into graphs of first column, mouse with pressed left button just moves the whole graph instead of selecting region. <Eduard> Duplicates (18) |
Comment by Alexei Vladishev [ 2012 Jun 19 ] |
(4) I have 20x20 screen filled with graphs with 10 seconds refresh interval set in user profile. Firefox keeps loading this screen forever while the screen (header, footer, filter are ok) stays blank, however I see huge number of requests coming to Apache. I see no graphs displayed after waiting for around 10 minutes. Zabbix 2.0.0 can handle this situation easily displaying all 400 graphs after a few seconds. Same happens under Configuration. The table is quickly displayed with 'Change' links, but no actual graphs displayed. I see that front-end tries to refresh cells every 10 seconds. Not sure if it has to be done this way when configuring a screen. PS Increasing of the refresh to 60 seconds interval fixes displaying, however it is still very slow. Also see (5). |
Comment by Alexei Vladishev [ 2012 Jun 19 ] |
(5) Doing more experiments with 20x20 screen. Apache shows that all graphs are updated after 60 seconds, but I see old images with older timestamp. It could be something related to caching, adding a sequence as a GET variable will probably help. Perhaps there are other solutions. <Eduard> Moved to |
Comment by Alexey Fukalov [ 2012 Jul 04 ] |
(6) <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 04 ] |
(7) <Eduard> Duplicates (1) <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 04 ] |
(8) <Eduard> RESOLVED <Vedmak> Sorting now works, but after each sorting change it seems to start new update process. For example we have interval 10 seconds, after load i waited 5 sec. and changed sorting, new screen element is updated every 5 sec. <Eduard> RESOLVED <Vedmak> When screen is opened, sorting reloads whole page, but after first refresh sorting reload only sorted element. <Eduard> RESOLVED. Now sorting work like in Host Triggers <Vedmak> sorting by time is missing <Eduard> RESOLVED <Vedmak> In sort selection list, all fields first go with 'ascending' option, only Time has incorrect order. <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 04 ] |
(9) <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 04 ] |
(10) <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 04 ] |
(11) <Eduard> RESOLVED <Vedmak> still reproducible <Eduard> RESOLVED <Vedmak> still reproducible <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 04 ] |
(12) <Eduard> RESOLVED <Vedmak> not sure if it's related, but... Seems data overview is not updated, numbers are not changed. <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 04 ] |
(13) <Eduard> RESOLVED <Vedmak> Same for monitoring->graphs <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 04 ] |
(14) <Eduard> RESOLVED <Vedmak> Now when timeline is set to "Now" page sends 2 identical ajax requests. <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 04 ] |
(15) <Eduard> Sorting jumps because of changing $page['file'] variable. If we decide to move sorting to configuration this issue will disappears. <Eduard> RESOLVED, see (8) <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 04 ] |
(16) <Eduard> RESOLVED <Vedmak> works if map is current screen element, but if we have screen with screen with map, popups don't work. <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 04 ] |
(17) <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 04 ] |
(18) <Eduard> RESOLVED <Vedmak> zooming works now, but not always synchronized. <Eduard> RESOLVED <pavels> CLOSED. |
Comment by Alexey Fukalov [ 2012 Jul 04 ] |
(19) <Eduard> RESOLVED <Vedmak> still reproducible <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 04 ] |
(20) <Eduard> RESOLVED <Vedmak> popup that is shown on hover now removed on refresh, but if we click on element to get static popup, it's not removed on refresh. <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 04 ] |
(21) Also new option 'hostids' should be documented. I couldn't really understand what it does, get options should filter reasults, but this one seems only add additional field to result. I'm not sure it should be in screen items class at all. <pavels> Let's discuss this together when you reach this point. <Eduard> RESOLVED, created new API class - CTemplateScreenItem <Vedmak> CLOSED complete implementation is moved to |
Comment by Alexey Fukalov [ 2012 Jul 04 ] |
(22) <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 04 ] |
(23) <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 04 ] |
(24) <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 04 ] |
(25) <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 04 ] |
(26) This branch seems change latest data history view too, is there any specs what is changed? <Eduard> RESOLVED <richlv> hmm, i believe the main point here was "what is changed" - exact description is utterly missing -> REOPENED <Eduard> RESOLVED <richlv> documentation part split out as (68), CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 04 ] |
(27) <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 04 ] |
(28) New classes lack phpdoc. <pavels> All of the properties and methods should be commented. <pavels> The phpdoc comments lack parameter and return value descriptions. Please add them according to the guidelines https://zabbix.org/wiki/Docs/specs/coding_style#PHPDoc <Eduard> RESOLVED <pavels> CLOSED. |
Comment by Pavels Jelisejevs (Inactive) [ 2012 Jul 06 ] |
(29) When I change the interval on the timeline, the graph scale is not updated. That is, if an interval of 1h has been selected on the timebar, and I change it to 1 month, the whole graph interval will still be 1h. <Eduard> Timeline box increasing width, commonly, starting from 8 days. Work same as trunk. WON'T FIX. <pavels> Sorry, this issue seems to be a duplicate of (1) and (7). CLOSED. |
Comment by Pavels Jelisejevs (Inactive) [ 2012 Jul 09 ] |
(30) The properties of the new classes should use camel case and probably shouldn't be public. <Eduard> RESOLVED <pavels> Please add a phpdoc to all properties. <pavels> Please documents the properties themselves too. <pavels> CLOSED. |
Comment by Pavels Jelisejevs (Inactive) [ 2012 Jul 09 ] |
(31) You don't need to set default option values in jsrpc.php in screen.get, because they are set in the screen classes. Also, to avoid all of the isset blocks, the options can be passed as an array in GET and then passed directly to the screen class. <Eduard> RESOLVED <pavels> CLOSED. |
Comment by Alexey Fukalov [ 2012 Jul 12 ] |
(32) <Eduard> RESOLVED <Vedmak> still reproducible <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 13 ] |
(33) <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Pavels Jelisejevs (Inactive) [ 2012 Jul 13 ] |
(34) When I open monitoring - graphs, no graphs are displayed, and I see an error: Undefined offset: 0 [charts.php:199] <Eduard> RESOLVED <pavels> The error should be displayed as plain text, not as an image. <Eduard> RESOLVED <pavels> The message should be changed to "No graphs defined". We don't use the word "chart" anywhere. <Eduard> RESOLVED <pavels> CLOSED. |
Comment by Alexey Fukalov [ 2012 Jul 16 ] |
(35) <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 16 ] |
(36) <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Stefan [ 2012 Jul 18 ] |
maybe a redseign with html5?! or ajax?! <Eduard> We must to support IE8 browser and others.., so at this time we not ready to use HTML5.. But it probably will happens in future! WON'T FIX. |
Comment by Alexey Fukalov [ 2012 Jul 18 ] |
(37) <Eduard> RESOLVED <pavels> The problem remains in Chrome. I can more the timeline up to 31.07.2012 00:00 by dragging it to the border. <Eduard> Please retest it again.. <pavels> Moved to |
Comment by Alexey Fukalov [ 2012 Jul 18 ] |
(38) <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 18 ] |
(39) When screen is opened in monitoring and then changed in another tab or by another user, these changes are not displayed in opened screen after automatic refresh. <Eduard> SPEC: Flicker-free screens don't refresh himself configuration, only update data. WON'T FIX. |
Comment by Alexey Fukalov [ 2012 Jul 18 ] |
(40) Url screen elements are not refreshed by automatic refresh. <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 18 ] |
(41) <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 19 ] |
(42) <Eduard> RESOLVED <Vedmak> still reproducible <Eduard> Every screen remember himself stime and period, if it was set to "Now" next time it will open on the remembered place. WON'T FIX. <pavels>This seems to be a duplicate of (47). It contains a more detailed problem description. CLOSED. |
Comment by Alexey Fukalov [ 2012 Jul 19 ] |
(43) <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 19 ] |
(44) Same if you click on 'Ack' link to go to acknowledge form and there click cancel. <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 19 ] |
(45) <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 19 ] |
(46) <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Kenneth Palmertree [ 2012 Jul 22 ] |
(47) I know this is still in development but want to note this in case it was not noticed already. In the latest version For example: Tested this in DM setup of Zabbix <Eduard> Please retest with latest dev branch.. <pavels> Still reproducible. <Eduard> RESOLVED, please retest (related with (42)) <pavels> CLOSED. |
Comment by richlv [ 2012 Jul 24 ] |
kenneth, it is very important to have early feedback as it allows to get the feature in usable state earlier and saves time later - thanks for testing |
Comment by richlv [ 2012 Jul 24 ] |
(48) looks like this branch ends up affecting other areas (latest data and web monitoring were mentioned in comments). unless i missed it, there is no list of "expected to be affected" pages and/or functionality, which would make testing very hard. a full list such pages/functionality areas should be produced. <Eduard> RESOLVED. Spec says: "To use flicker-free screens the following pages shall be redesigned: Monitoring->Screen, Monitoring->Slide shows, Monitoring->Graphs, Monitoring->Latest data->History, Monitoring->Latest data->Host screens." |
Comment by Alexey Fukalov [ 2012 Jul 24 ] |
(49) <Eduard> It's ok, WON'T FIX <pavels> No, this seems to be a bug. REOPENED. <Eduard> RESOLVED <pavels> CLOSED. |
Comment by Alexey Fukalov [ 2012 Jul 24 ] |
(50) <Eduard> RESOLVED <pavels> Still reproducible in Firefox. <Eduard> RESOLVED, related to (18) and (42) <pavels> CLOSED. |
Comment by Alexey Fukalov [ 2012 Jul 25 ] |
(51) in Monitoring->events timeline stuck after first move. <Eduard> RESOLVED <Vedmak> CLOSED |
Comment by Alexey Fukalov [ 2012 Jul 30 ] |
(52) <Eduard> RESOLVED <pavels> CLOSED. |
Comment by Alexey Fukalov [ 2012 Jul 30 ] |
(53) <Eduard> RESOLVED <pavels> CLOSED. |
Comment by richlv [ 2012 Aug 01 ] |
(54) dev branch rev 29354 - debug output has been committed in simple graphs period=1h, (3600) mm, make that all pages with time period scrollbar <Eduard> RESOLVED =))) <richlv> CLOSED |
Comment by richlv [ 2012 Aug 01 ] |
(55) issue title and summary talk about configuration as well, which doesn't seem to be implemented. which is correct ? <Eduard> Configuration of screens work without flicker-free self refresh.. <richlv> modified description & summary a bit more, CLOSED |
Comment by richlv [ 2012 Aug 01 ] |
(56) have screen with dynamic elements, open it in monitoring. change time period displayed, switch host - time period is reset back to 1h and "now" <Eduard> RESOLVED <richlv> CLOSED |
Comment by richlv [ 2012 Aug 01 ] |
note : a regression regarding dynamic screens has been filed as |
Comment by richlv [ 2012 Aug 02 ] |
(57) dev branch rev 29364; opera 12.0, firefox 14.0.1 and konqueror 4.7.2 - grab the rightmost edge of the scrollbar and drag it to the right. result : <Eduard> RESOLVED <richlv> now, whenever that is done when the period is already at "now", period length is increased (leftmost edge moves) - period should not change in this case <Eduard> Moved to |
Comment by Pavels Jelisejevs (Inactive) [ 2012 Aug 02 ] |
(58) I have a screen containing a screen with a graph. The position of the timeline for this screen is not remembered, it's always 1h "now". <Eduard> RESOLVED <pavels> Now if i select a time interval on the graph, then change it on the timeline, leave the screen and come back, it is not remembered. Tested in firefox. <Eduard> RESOLVED <pavels> CLOSED. |
Comment by Pavels Jelisejevs (Inactive) [ 2012 Aug 02 ] |
(59) Monitoring - screens and graphs does not work in IE8. <Eduard> RESOLVED <pavels> CLOSED. |
Comment by Pavels Jelisejevs (Inactive) [ 2012 Aug 02 ] |
(60) In IE9 graph zoom works only once. <Sasha> on my IE9 9.0.8112.16421 zoom works as expected. <pavels> Hmm, may be it's our VM issue. Ok, CLOSED. <Sasha> REOPENED moved to |
Comment by Alexander Vladishev [ 2012 Aug 03 ] |
(61) content are not updated when change time from (now!) by a calendar or scroll bar components <Sasha> on IE9 works fine without the problem <Eduard> RESOLVED <Sasha> CLOSED |
Comment by Alexander Vladishev [ 2012 Aug 06 ] |
(62) zoom show latest data on graph; time navigator works correctly <Sasha> on IE9 works fine without the problem <Eduard> RESOLVED <Sasha> CLOSED |
Comment by Alexander Vladishev [ 2012 Aug 06 ] |
(63) after move a scroll bar to a left position periods "2h", "3h"... are blocked <Eduard> Moved to |
Comment by Alexander Vladishev [ 2012 Aug 06 ] |
(64) after network error elements of a screen cease to be updated <Eduard> Moved to |
Comment by Alexander Vladishev [ 2012 Aug 06 ] |
(65) Elements of the time navigator aren't updated
<Eduard> Moved to |
Comment by Pavels Jelisejevs (Inactive) [ 2012 Aug 07 ] |
(66) Sometimes when I select an interval on the timeline and refresh the page, the timeline bar is pushed back 1 hour. <Eduard> Moved to |
Comment by Pavels Jelisejevs (Inactive) [ 2012 Aug 07 ] |
(67) There seems to be some debug code in CScreenGraph::get(): "echo strval(false);" <Eduard> RESOLVED <pavels> CLOSED. |
Comment by richlv [ 2012 Aug 07 ] |
(68) didn't see it linked before - spec is at https://zabbix.org/wiki/Docs/specs/ZBXNEXT-1253
<Eduard> Moved to |
Comment by Pavels Jelisejevs (Inactive) [ 2012 Aug 07 ] |
(69) A couple of problems with the "updateProfile" parameter: <Eduard> RESOLVED <pavels> I suggest we don't compare values to "true" or "false", but cast them to integers when sending AJAX requests from javascript. <Eduard> RESOLVED <pavels> CLOSED. |
Comment by richlv [ 2012 Aug 07 ] |
(70) db patch should update any changed 'profiles' contents <Eduard> RESOLVED, (profiles.sql) <Sasha> CLOSED <richlv> patch should be also added for 2.2, REOPENED <Sasha> patches for 2.2 should be added in trunk only! <richlv> 2.2 patches seem to be missing <Eduard> RESOLVED in <Sasha> CLOSED |
Comment by Alexander Vladishev [ 2012 Aug 08 ] |
Successfully tested! |
Comment by Eduards Samersovs (Inactive) [ 2012 Aug 08 ] |
Fixed in versions pre-2.1.0 (beta) r29459, pre-2.0.3 r29458 |
Comment by Oleksii Zagorskyi [ 2012 Oct 23 ] |
Fix version: 2.2.0 -> 2.1.0 |
Comment by richlv [ 2013 Aug 12 ] |
this resulted in a regression for log/text file viewing : |