Ticket #4449 (closed defect (bug): duplicate)

Opened 5 years ago

Last modified 3 years ago

Kubrick ordered list problem for IE7

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

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

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

Change History

  • Milestone changed from 2.2.1 to 2.2.2
  • 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?

neoen5 years ago

Differences between IE7 and Firefox

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

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

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

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

You might be able to do something like this:

* html .post ul {
    list-style-type: disc;
}
  • Status changed from closed to reopened
  • Resolution invalid deleted
  • Milestone set to 2.2.2

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

  • Milestone changed from 2.2.2 to 2.3 (trunk)

comment:9   ryan4 years ago

  • Milestone changed from 2.3 to 2.4

comment:10   djr4 years ago

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

comment:11   djr4 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   djr4 years ago

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

djr4 years ago

comment:13   djr4 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";
	}

ninjaWR3 years ago

patch (from r9837) of djr's fix

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

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 ;)

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

Patch - valid CSS

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

See also #4518

  • Keywords changed from ie7 has-patch needs-feedback to has-patch needs-feedback ie7
  • 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.

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

Duplicate: 4518

*ups* Duplicate: #4518

Note: See TracTickets for help on using tickets.