• Do not register here on develop.twiki.org, login with your twiki.org account.
• Use View topic Item7848 for generic doc work for TWiki-6.1.1. Use View topic Item7851 for doc work on extensions that are not part of a release. More... Close
• Anything you create or change in standard webs (Main, TWiki, Sandbox etc) will be automatically reverted on every SVN update.
Does this site look broken?. Use the LitterTray web for test cases.

Make a new table in WYSIWYG

Start writing in a cell

Hit RETURN to start a new line.

Note that the first line you wrote jumps down a little bit.

Write some text in the 2nd line you just wrote

Save the topic.

View the TML

Note that the WYSIWYG plugin has added an unwanted <P /> in front of the first line.

You can remove it in TML mode but not in WYSIWYG mode.

KJL

I can't reproduce this in FF 1.04, FF 1.5 or IE 6, following the above instructions. Guessing it's an artifact of Kupu 1.2 that has been fixed in 1.3.2

CC

The tables behaves differently in IE and FF.

The problem I reported is EASY to reproduce with IE. Very very easy. And it also does it here on develop. I do not use an old kupu when I have an up to date SVN installation I assume.

Please try again. The bug is there.

See WysiwygTableIssue

KJL

Ah; assuming the bug I am seeing is the bug you are seeing, then that is not a WysiwygPlugin bug, nor even a Kupu bug; it is a browser bug. Bill's Browser is Broken Again.

Not much I can do about it, sorry.

CC

Maybe Bills browser is Broken. But that is what people have. Discarding does not make the problem go away.

I do not see a great need for people to put a <P /> as the first string in a table cell. I suggest you strip it off then when the topic is saved. The regex should look for the string right at the beginning of a table cell and strip it off. Making line breaks in tables is quite common and with the current problem quite many people will be very annoyed by this problem.

If people want empty lines in front they will probably use BR tags instead. I am sure you will solve more problems with this workaround than you will create. Naturally such a work around will not prevent the text from moving down when you edit but at least people will not have to re-edit in TML mode to get rid of the unwanted empty lines.

KJL

sigh I'm very reluctant to put in "special fixes" for browsers, because of the flak it is bound to cause. For example: user starts with a cell containing only <p/>. Edit in Kupu, save, strip <p/> and the cell disappears because it has no content.

We need a total solution for table cells. Perhaps this technique has a place in it; feedback in Item1231 please.

CC

I understand your reluctance. But sometimes you need silly solutions to work around bugs in other people's software. In this case Billy-Boys browser.

I doubt many will key < p/>. They may press the enter button as the first action and that probably gives a <p> </p> sequence. Or a <BR>. But it is quite normal that people write some text, hit return to make a new line or some white space and write some more text. And this is where the very annoying problem happens with the extra <p />

I do not understand the connection to 1231 confused

KJL

The connection to 1231 is the need for a global solution to tables. this problem is just one that relates to that.

CC

After careful thought, I think we have to let this stand. There is no way for the plugin to recognise when a P was added "in error", and for it to strip out Ps in table cells would be a lot more irritating that the current situation.

Discarded, because I have no way to pass a bug on to IE and M$ would claim it's a feature anyway.

CC

Well. I think discarding this bug is the wrong decision. "You can lead the horse to the water but you cannot force it to drink". So I will not reopen this. But I am sure you will see this bug reported again once the plugin becomes part of Dakar - because it makes the plugin close to useless when editing tables in WYSIWYG in IE.

KJL

ItemTemplate
Summary WYSIWYG table. Making a new line inside table cell adds unwanted P / before first line
ReportedBy KennethLavrsen
Codebase

SVN Range Wed, 14 Dec 2005 build 7851
AppliesTo Extension
Component WysiwygPlugin
Priority Normal
CurrentState No Action Required
WaitingFor

Edit | Attach | Watch | Print version | History: r13 < r12 < r11 < r10 < r9 | Backlinks | Raw View | Raw edit | More topic actions
Topic revision: r13 - 2006-01-16 - KennethLavrsen
 
This site is powered by the TWiki collaboration platform Powered by PerlCopyright © 2008-2018 by the contributing authors. All material on this collaboration platform is the property of the contributing authors.
Ideas, requests, problems regarding TWiki? Send feedback