Opened 15 years ago
Closed 14 years ago
#17980 closed defect (bug) (fixed)
IE6 warning and missing image + change string
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 3.3 | Priority: | lowest |
| Severity: | normal | Version: | 3.2 |
| Component: | General | Keywords: | has-patch |
| Focuses: | Cc: |
Description
There is no icon of Internet Explorer visible in Dashboard warning when using IE6 (see attached screenshot). IE7 and IE8 works well...
Also one string should be changed, there is paragraph tag included which should not be there (or I did not find similar strings - usually paragraph tags are outside translatable string). See attached patch.
Attachments (5)
Change History (12)
#2
@
15 years ago
Sorry, I added ending paragraph tag and forgot removing it from original string. Patch updated, thank you.
Yes, I know that WP 3.2 will not support IE6, but showing this informative box about insecure browser should be IE6 friendly. But it is not a big problem, I guess...
#3
in reply to:
↑ 1
@
15 years ago
Replying to nacin:
The string has both <p> and </p> in it. You're right we should remove them, but not until 3.3. Your patch removes <p> but leaves </p>.
Missing IE image, now that's a shame :-) Wondering where it ran off to? Probably a CSS issue.
your wrong
its a typical #blaimNacin thing
#5
@
14 years ago
- Milestone changed from Awaiting Review to 3.3
- Owner set to nacin
- Status changed from new to accepted
#6
@
14 years ago
17980.patch also fixes the font color and icon position (see the screenshot). IE6 doesn't display PNG alpha transparency properly, but I guess it's not worth hacking.
The string has both <p> and </p> in it. You're right we should remove them, but not until 3.3. Your patch removes <p> but leaves </p>.
Missing IE image, now that's a shame :-) Wondering where it ran off to? Probably a CSS issue.