Ticket #4890 (closed Bugs: fixed)

Opened 2 years ago

Last modified 2 years ago

Advanced Settings page not compatible with Internet Explorer 8

Reported by: bae388 Assigned to: mbrevda
Priority: minor Milestone: 2.9
Component: Internet Explorer (provide version) Version: 2.8-branch
Keywords: IE8, Advanced Settings Cc:
Confirmation: Need testing Distro:
Backend Engine: Asterisk 1.8 Distro Ver:
Backend Ver: SVN Revision (if applicable):

Description

The green checkmark buttons on the "Advanced Settings" page are not clickable while using Internet Explorer 8. Also, all of the green checkmark buttons are visible without having changed any of the settings. I assume they are only supposed to be visible if a field was changed. If a green checkmark button is clicked, nothing happens (no loading icon), and the "Apply Configuration Settings" option does not appear.

I have tested this on two machines running Windows XP SP3 and Windows Vista (both with IE8). When I used Mozilla Firefox from the same machines, the green checkmark buttons worked fine, and the settings were able to be applied successfully.

Change History

02/28/11 10:18:07 changed by p_lindheimer

  • owner set to mbrevda.
  • priority changed from regression to minor.

Moshe,

can you have a look and see if you see anything obvious that may be doing this, given both all the green submits are showing AND the ajax call never fires, it seems like something might be killing the js on that page all together???

02/28/11 10:34:08 changed by mbrevda

Do you know of how to debug js in IE? IE isnt throwing any errors and I cant think of why it should fail.

02/28/11 10:49:04 changed by p_lindheimer

no idea how, was hoping maybe you knew some tricks??? What's odd in looking at the code, is the initial hiding is not js, it's from core.csss I believe: 

.save, .adv_set_default{border-style: none;}
.savetd{display:none;}

The .savetd{display:none;} and there is nothing special about that???

02/28/11 11:26:01 changed by mbrevda

While we could try breaking the first line in to two, I'm pretty sure thats written according to spec... Oh right, IE doesnt follow the spec :(

02/28/11 11:57:32 changed by p_lindheimer

it's the second line that does not appear to be taking, it's not clear from the reported ticket if the first line is also a problem.

02/28/11 13:54:22 changed by mbrevda

Yeah, but if the first line doesnt work, the parser might halt or something

03/01/11 10:27:05 changed by p_lindheimer

  • confirmation changed from Unreviewed to Need testing.

any chance of testing this on IE9 to see if the same issue exists there?

Moshe, can you test by removing: 

.alt_table tr:nth-child(even){background-color: #FCE7CE;}

from mainstyle.css (and make sure cache is flushed). That is not supported in IE8 for sure, you wouldn't think it would fail because of that but it's an easy check.

03/01/11 12:34:16 changed by mickecarlsson

In IE8, press F12 for Developer mode

03/01/11 12:57:25 changed by mbrevda

@mickecarlsson - great find! @p_lindheimer - no discernable difference

03/01/11 15:59:19 changed by mbrevda

  • status changed from new to closed.
  • resolution set to fixed.

(In [11636]) should fix #4890 - IE SUCKS! IE SUCKS! IE SUCKS! IE SUCKS!