Skip to main content


Eclipse Community Forums

      Home
Home » Eclipse Projects » Nebula » [RichTextPainter] IllegalArgumentException in ResetSpanStylePaintInstruction
[RichTextPainter] IllegalArgumentException in ResetSpanStylePaintInstruction [message #1743513] Fri, 16 September 2016 02:52 Go to next message
Eclipse UserFriend
Whenever I have nested span elements like in the following HTML:
Quote:
<span style="color:rgb(255, 0, 0)"><span style="color:rgb(210, 105, 30)">ERROR, STOP_FTC:SAS_Startup failed for ODN_SAS:Ground_Common: COMMUNICATION_ERROR: Error on the communication channel</span></span>


The painter throws an IllegalArgumentException:
Quote:
Caused by: java.lang.IllegalArgumentException: Argument cannot be null
at org.eclipse.swt.SWT.error(SWT.java:4472)
at org.eclipse.swt.SWT.error(SWT.java:4406)
at org.eclipse.swt.SWT.error(SWT.java:4377)
at org.eclipse.swt.graphics.GC.setForeground(GC.java:3513)
at org.eclipse.nebula.widgets.richtext.painter.instructions.ResetSpanStylePaintInstruction.paint(ResetSpanStylePaintInstruction.java:40)


The problem is that the ResetSpanStylePaintInstruction sets the state color to null instead to the color of the parent span. So when there are two adjacent </span> elements, the first one sets the state color to null and the second one then sets the gc foreground to null.

Shall I file a bug for this, or am I doing something wrong?
Re: [RichTextPainter] IllegalArgumentException in ResetSpanStylePaintInstruction [message #1743518 is a reply to message #1743513] Fri, 16 September 2016 03:41 Go to previous messageGo to next message
Eclipse UserFriend
Well looking at your html snippet I wonder why you have a nested span? The outer span is kind of useless.

But since it is valid html feel free to create a ticket for this.
Re: [RichTextPainter] IllegalArgumentException in ResetSpanStylePaintInstruction [message #1743521 is a reply to message #1743518] Fri, 16 September 2016 03:49 Go to previous messageGo to next message
Eclipse UserFriend
Just checked the implementation. In TagProcessingState I commented that stacking of colors is not necessary. IIRC the ckeditor does not produce such combinations. But I could be wrong.

The solution would be to introduce the same stacking behaviour as for fonts.
Re: [RichTextPainter] IllegalArgumentException in ResetSpanStylePaintInstruction [message #1743527 is a reply to message #1743521] Fri, 16 September 2016 04:33 Go to previous messageGo to next message
Eclipse UserFriend
The problem is that the html formatting directives are inserted into normal text by markup display converters (working with user defined regular expressions). Since I have no control over the converters defined by the users, I somehow have to make this work. One possibility obviously would be to remove all superfluous markup before passing the text to the painter, but that would not fix all cases. If the HTML looked like this:
Quote:
<span style="color:rgb(255, 0, 0)">lots of text with red color in a the top span<span style="color:rgb(210, 105, 30)">some more text with a different color</span></span>

removing spans would no longer work.

[Updated on: Fri, 16 September 2016 04:35] by Moderator

Re: [RichTextPainter] IllegalArgumentException in ResetSpanStylePaintInstruction [message #1743532 is a reply to message #1743527] Fri, 16 September 2016 04:55 Go to previous messageGo to next message
Eclipse UserFriend
so file a bug and we introduce stacking also for colours. ckeditor is closing a color span before applying a new one. Therefore it is implemented as it is.
Re: [RichTextPainter] IllegalArgumentException in ResetSpanStylePaintInstruction [message #1743534 is a reply to message #1743532] Fri, 16 September 2016 05:08 Go to previous messageGo to next message
Eclipse UserFriend
Thanks for looking into this, I have written a bug report: https://bugs.eclipse.org/bugs/show_bug.cgi?id=501568
Re: [RichTextPainter] IllegalArgumentException in ResetSpanStylePaintInstruction [message #1743560 is a reply to message #1743534] Fri, 16 September 2016 07:43 Go to previous messageGo to next message
Eclipse UserFriend
I provided a patch that needs to be checked. Is it possible for you to checkout and test it before I merge it into the repository? If we are lucky it can be part of the 1.1.0 release of Nebula next week then.
Re: [RichTextPainter] IllegalArgumentException in ResetSpanStylePaintInstruction [message #1743604 is a reply to message #1743560] Fri, 16 September 2016 15:38 Go to previous messageGo to next message
Eclipse UserFriend
A Gerrit P2 repo can be found here:

https://hudson.eclipse.org/nebula/job/nebula.stable.gerrit/lastSuccessfulBuild/artifact/releng/org.eclipse.nebula.updatesite/target/repository/
Re: [RichTextPainter] IllegalArgumentException in ResetSpanStylePaintInstruction [message #1743694 is a reply to message #1743604] Mon, 19 September 2016 01:41 Go to previous message
Eclipse UserFriend
Thanks you for the repo url. I have tested the fix and it works for my cases.
Previous Topic:news.eclipse.org is shutting down.
Next Topic:[Richtext Editor] How to wait to load of browser
Goto Forum:
  


Current Time: Thu Mar 20 20:07:57 EDT 2025

Powered by FUDForum. Page generated in 0.05383 seconds
.:: Contact :: Home ::.

Powered by: FUDforum 3.0.2.
Copyright ©2001-2010 FUDforum Bulletin Board Software

Back to the top

Sign up to our Newsletter

A fresh new issue delivered monthly