WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 5 years ago

Last modified 5 years ago

#4449 closed defect (bug) (duplicate)

Kubrick ordered list problem for IE7

Reported by: neoen Owned by: djr
Milestone: 2.8 Priority: low
Severity: normal Version: 2.2
Component: Template Keywords: has-patch needs-feedback
Focuses: Cc:

Description

Hello, in default Kubrick theme there is a problem with ordered list (<ul>
<li></li></ul>) in IE 7 which is not appearing... In another theme it works well, so I suppose that there is some problem in css.
Thank you for your help...

Attachments (4)

IE7_Firefox.PNG (42.5 KB) - added by neoen 7 years ago.
Differences between IE7 and Firefox
4449.diff (377 bytes) - added by djr 6 years ago.
4449-style.css.diff (661 bytes) - added by ninjaWR 5 years ago.
patch (from r9837) of djr's fix
4449-vk.diff (478 bytes) - added by vladimir_kolesnikov 5 years ago.
Patch - valid CSS

Download all attachments as: .zip

Change History (25)

comment:1 foolswisdom7 years ago

  • Milestone changed from 2.2.1 to 2.2.2

comment:2 rob1n7 years ago

  • Keywords reporter-feedback ie7 added
  • Milestone changed from 2.2.2 to 2.4 (future)

Can you provide a description of what's wrong? Or someone else with IE 7?

neoen7 years ago

Differences between IE7 and Firefox

comment:3 neoen7 years ago

In Firefox there is ordered list, but in IE7 not (missing). I use default Kubrick (Default 1.6) theme without any changes. Please see attached .png file...

comment:4 JeremyVisser7 years ago

  • Milestone 2.4 (future) deleted
  • Resolution set to invalid
  • Status changed from new to closed

This is an IE bug, and comes from not supporting the :before pseudo-class.

comment:5 neoen7 years ago

So, this cannot be repaired somehow, for example in css? It is default Wordpress theme and it is very annoying...

comment:6 JeremyVisser7 years ago

You might be able to do something like this:

* html .post ul {
    list-style-type: disc;
}

comment:7 neoen7 years ago

  • Milestone set to 2.2.2
  • Resolution invalid deleted
  • Status changed from closed to reopened

I tried to add this code to style.css, but this does not work.
But I found there following:
/* Begin Lists

Special stylized non-IE bullets
Do not work in Internet Explorer, which merely default to normal bullets. */

So there are special bullets for non IE7 browsers, but in IE7 there shuld be normal bullets. But in IE7 there are no bullets :-(
Thank you for your help...

comment:8 rob1n7 years ago

  • Milestone changed from 2.2.2 to 2.3 (trunk)

comment:9 ryan7 years ago

  • Milestone changed from 2.3 to 2.4

comment:10 djr6 years ago

  • Cc djr added
  • Owner changed from anonymous to djr
  • Status changed from reopened to new

comment:11 djr6 years ago

Let's hope I did this correctly. I attached a patch which specifically targets IE7. I realise this adds a lot of extra css to default.css, but it's the only way to have bullets across all browsers. I think it's better to move to image based bullets or include a seperate stylesheet for IE7, but nevertheless my patch works.

Tested with no side-effects: IE7/Win, FF2/Win, FF2/Mac, Safari3/Mac. A temporary demo can be seen here: http://www.iphonemobiel.nl.

comment:12 djr6 years ago

  • Keywords has-patch needs-feedback added; reporter-feedback removed

djr6 years ago

comment:13 djr6 years ago

Ok, I s*ck at making patches, so either ignore or please fix.
Here's the code block to be included:

/* Fix for IE7 not showing bullets */
html>body .entry ul {
	*margin-left: 0px;
	*padding: 0 0 0 30px;
	*list-style: disc;
	*padding-left: 10px;
	*text-indent: 10px;
	}

html>body .entry ul li {
	*margin: 7px 0 8px 10px;
	}
/* End fix for IE7 */

This code block need to go just below:

.entry ul li:before, #sidebar ul ul li:before {
	content: "\00BB \0020";
	}

ninjaWR5 years ago

patch (from r9837) of djr's fix

comment:14 ninjaWR5 years ago

  • Milestone changed from 2.9 to 2.8
  • Priority changed from normal to low

would be nice if this were to make it into 2.8, since I'm sure the posts look weird to visitors using IE... I wish IE were more compliant, but don't we all ;)

comment:15 vladimir_kolesnikov5 years ago

Have a better patch that does the same but is still valid CSS. Tragetted for IE7.

vladimir_kolesnikov5 years ago

Patch - valid CSS

comment:16 jacobsantos5 years ago

I've always wondered why there weren't any styles on the theme when using IE.

comment:17 mrmist5 years ago

See also #4518

comment:18 jacobsantos5 years ago

  • Keywords changed from ie7 has-patch needs-feedback to has-patch needs-feedback ie7

comment:19 rowoot5 years ago

  • Summary changed from Kubrick ordered list problem to Kubrick ordered list problem for IE7

Tested patch 4449-style.css.diff, 4449-vk.diff and 4449.diff.
None seem to work in IE7 as of now. Tried it out with the latest nightly build.

I`ll try working on a update to these patches.

comment:20 Nicholas915 years ago

  • Keywords ie7 removed
  • Resolution set to duplicate
  • Status changed from new to closed

Duplicate: 4518

comment:21 Nicholas915 years ago

*ups* Duplicate: #4518

Note: See TracTickets for help on using tickets.