[ZBX-3244] dropdown changes its style when navigating menu Created: 2010 Nov 29  Updated: 2017 May 30  Resolved: 2011 Nov 23

Status: Closed
Project: ZABBIX BUGS AND ISSUES
Component/s: Frontend (F)
Affects Version/s: 1.8.4rc2
Fix Version/s: 1.8.10, 2.0.0

Type: Incident report Priority: Trivial
Reporter: Aleksandrs Saveljevs Assignee: Unassigned
Resolution: Fixed Votes: 0
Labels: reports, trivial, usability
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Attachments: PNG File mode-style-1.png     PNG File mode-style-2.png    

 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.



 Comments   
Comment by richlv [ 2010 Nov 29 ]

quite wicked. happens also in opera 10.11

Comment by Alexander Sorokin (Inactive) [ 2011 Nov 23 ]

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

Comment by Aleksandrs Saveljevs [ 2011 Nov 23 ]

(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.

Comment by Alexander Sorokin (Inactive) [ 2011 Nov 30 ]

Fixed in 1.8.10rc1 and trunk

Generated at Fri Mar 29 16:22:23 EET 2024 using Jira 9.12.4#9120004-sha1:625303b708afdb767e17cb2838290c41888e9ff0.