Item1987: Breaks TinyMCE CSS

Running trunk NatSkin and TinyMCEPlugin.

Bullets aren't displayed, unless you append Eg. &skin=pattern to the edit URL.

-- PaulHarvey - 30 Aug 2009

Using Firefox 3.0.6, apparently this is the case with FF 3.5x as well.

The issue seems to be the reset of padding and margin properties in BaseStyles.css.

-- PaulHarvey - 30 Aug 2009

A screenshot says more than 1k words

-- MichaelDaum - 31 Aug 2009

So you can't read my mind? Oops. Here it is

-- PaulHarvey - 31 Aug 2009

• raw editor showing topic text:
  

• wysiwyg editor showing fault:
  

• wysiwyg editor showing hacked css without fault:
  

---

Okay. This is actually the result of a bad rendering of %NATSTYLEURL%, which gets assigned to %FOSWIKI_COLORS_URL% which in turn is used in TinyMCE's INIT settings.

Also this explains why I've been getting some HTTP Error 403's in my page views.

%NATSTYLEURL%

• Expected: /pub/System/JazzyNoteTheme/JazzyNoteStyle.css
• Actual: /pub/System/JazzyNoteTheme

Attached a diff to fix this problem.

• natskin4759_styleurl.diff: fix rendering of NATSTYLEURL, use URLPATH instead of PATH

-- PaulHarvey - 03 Sep 2009

Excellent

-- MichaelDaum - 03 Sep 2009

Main.MichaelTempest reports the fix works on IE8, Opera 10 and FF 3.52

I set upgraded to urgent, because I thought this might be a small enough patch that we could get it commited soon for the 1.0.7 release?

-- PaulHarvey - 13 Sep 2009

Tested successfully in these browsers today:

• IE 6.0.2900.5512.xpsp_sp3_gdr.090206-1234 (XP)
• Firefox (Iceweasel) 3.0.12 (Linux)
• Firefox 3.5.3 (XP)
• Opera 10.00 (Linux)
• Chrome 2.0.172.43 (XP)
• Epiphany 2.22.3 (Linux)
• Safari 4.0.3 (XP)

Actually I've found that BaseStyle.css kills WYSIWYG experience for users editing tables horrible padding, margins, etc. set to 0... Working on this.

-- PaulHarvey - 14 Sep 2009

BaseStyle.css uses a css reset framework "borrowed" from common grid css frameworks. In the next NatSkin version all css for foswikiTables, their rows and cells will be applied to the related elements under mceContentBody. These didn't have any default css before other than what is applied by browsers themselves.

-- MichaelDaum - 14 Sep 2009

Michael, thank you very much for the NatSkin update, it makes everything look quite lovely

Tested successfully on all the browsers listed above.

-- PaulHarvey - 15 Sep 2009