ZABBIX BUGS AND ISSUES

dropdown changes its style when navigating menu

Details

  • Zabbix ID:
    Reviewed 2.0

Description

Using the GUI as guest, go to "Reports" -> "Availability report". Observe in mode-style-1.png the style of the "Mode" dropdown. Now, move the mouse to point at "Inventory". Observe in mode-style-2.png that the dropdown changed its style.

Activity

Hide
richlv added a comment -

quite wicked. happens also in opera 10.11

Show
richlv added a comment - quite wicked. happens also in opera 10.11
Hide
Alexander Sorokin added a comment -

Fixed in development branch svn://svn.zabbix.com/branches/dev/ZBX-3244

Show
Alexander Sorokin added a comment - Fixed in development branch svn://svn.zabbix.com/branches/dev/ZBX-3244
Hide
Aleksandrs Saveljevs added a comment - - edited

(1) tagName is not guaranteed to be all uppercase, see http://reference.sitepoint.com/javascript/Element/tagName .

<asaveljevs> Also, the new JavaScript code uses "notactive" CSS class, but it does not seem to be defined anywhere.

<sanja> RESOLVED

<asaveljevs> We use toLowerCase() in the rest of our code:

$ grepjs tagname | grepjs tagName | grep toUpperCase | grep -Ev 'scriptaculous|prototype'
./frontends/php/js/main.js: if (menu_cell.tagName.toUpperCase() != 'SELECT') {
$ grepjs tagname | grepjs tagName | grep toLowerCase | grep -Ev 'scriptaculous|prototype'
./frontends/php/js/class.calendar.js: if(this.timeobjects[0].tagName.toLowerCase() == 'input'){
./frontends/php/js/class.calendar.js: if(this.timeobjects[i].tagName.toLowerCase() == 'input'){
./frontends/php/js/class.calendar.js: if(this.timeobjects[0].tagName.toLowerCase() == 'input'){
./frontends/php/js/class.calendar.js: if(this.timeobjects[0].tagName.toLowerCase() == 'input'){
./frontends/php/js/class.calendar.js: if(this.timeobjects[1].tagName.toLowerCase() == 'input'){
./frontends/php/js/class.calendar.js: if(this.timeobjects[2].tagName.toLowerCase() == 'input'){
./frontends/php/js/class.calendar.js: if(this.timeobjects[3].tagName.toLowerCase() == 'input'){
./frontends/php/js/class.calendar.js: if(this.timeobjects[4].tagName.toLowerCase() == 'input'){
./frontends/php/js/class.cviewswitcher.js: switch(obj.tagName.toLowerCase()) {
./frontends/php/js/class.cviewswitcher.js: switch(obj.tagName.toLowerCase()) {
./frontends/php/js/class.cviewswitcher.js: switch(obj.tagName.toLowerCase()) {
./frontends/php/js/class.csuggest.js: if(is_null(row) || (row.tagName.toLowerCase() != 'tr') || !isset('id',row)) return true;

<sanja> RESOLVED

<pavels> CLOSED.

Show
Aleksandrs Saveljevs added a comment - - edited (1) tagName is not guaranteed to be all uppercase, see http://reference.sitepoint.com/javascript/Element/tagName . <asaveljevs> Also, the new JavaScript code uses "notactive" CSS class, but it does not seem to be defined anywhere. <sanja> RESOLVED <asaveljevs> We use toLowerCase() in the rest of our code: $ grepjs tagname | grepjs tagName | grep toUpperCase | grep -Ev 'scriptaculous|prototype' ./frontends/php/js/main.js: if (menu_cell.tagName.toUpperCase() != 'SELECT') { $ grepjs tagname | grepjs tagName | grep toLowerCase | grep -Ev 'scriptaculous|prototype' ./frontends/php/js/class.calendar.js: if(this.timeobjects[0].tagName.toLowerCase() == 'input'){ ./frontends/php/js/class.calendar.js: if(this.timeobjects[i].tagName.toLowerCase() == 'input'){ ./frontends/php/js/class.calendar.js: if(this.timeobjects[0].tagName.toLowerCase() == 'input'){ ./frontends/php/js/class.calendar.js: if(this.timeobjects[0].tagName.toLowerCase() == 'input'){ ./frontends/php/js/class.calendar.js: if(this.timeobjects[1].tagName.toLowerCase() == 'input'){ ./frontends/php/js/class.calendar.js: if(this.timeobjects[2].tagName.toLowerCase() == 'input'){ ./frontends/php/js/class.calendar.js: if(this.timeobjects[3].tagName.toLowerCase() == 'input'){ ./frontends/php/js/class.calendar.js: if(this.timeobjects[4].tagName.toLowerCase() == 'input'){ ./frontends/php/js/class.cviewswitcher.js: switch(obj.tagName.toLowerCase()) { ./frontends/php/js/class.cviewswitcher.js: switch(obj.tagName.toLowerCase()) { ./frontends/php/js/class.cviewswitcher.js: switch(obj.tagName.toLowerCase()) { ./frontends/php/js/class.csuggest.js: if(is_null(row) || (row.tagName.toLowerCase() != 'tr') || !isset('id',row)) return true; <sanja> RESOLVED <pavels> CLOSED.
Hide
Alexander Sorokin added a comment -

Fixed in 1.8.10rc1 and trunk

Show
Alexander Sorokin added a comment - Fixed in 1.8.10rc1 and trunk

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: