Attachments (3)
Change History (9)
by , 15 years ago
Attachment: | html-in-message.png added |
---|
comment:1 by , 15 years ago
The reason for this is that ExtendedDialogs displays message strings using a JMultilineLabel which does not support HTML. There are many possibilities to fix this, two easy ones and the hard one (add proper HTML support to JMulti…). I favor the latter but I lack the skills to do so (that code confuses me in ways I never deemed possible). I'll attach a patch that fixes the issue at hand, but probably this is not the best way to go.
by , 15 years ago
Attachment: | jmultiline.patch added |
---|
Patch is unnecessary large due to Eclipse reformatting the source code. I still do not get why we need braces around single statement blocks, but well.
comment:2 by , 15 years ago
Summary: | raw html in UI message → [PATCH] raw html in UI message |
---|
comment:3 by , 15 years ago
Just a note: I do not consider this patch to be a good solution. We want proper HTML support in a multiline label, but I'm not going to work on this.
by , 15 years ago
Attachment: | new_jmultiline.patch added |
---|
Well, apparently I did work on it. This is a quick hack after I read a tip on Sun's bug tracker. Labels do support line breaking, but one must set the preferred width. The new jmutliline class derives this value automatically from the parent window. The patch is not well tested, there might be problems. Requires ExtendedDialog to not use a JScrollpane because it will display scrollbars even when not needed (they arent taken into account, might be fixable though). I didn't check if the max-width function actually works. Didn't check for other consumers, but it should have the same function names. Maybe I'll fix issues in my tree if I find them, but guarantees that this ever becomes a good patch.
comment:4 by , 15 years ago
Owner: | changed from | to
---|
Please checking yourself when you think it works as expected.
comment:5 by , 15 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:6 by , 15 years ago
Well, here goes. It's another new approach that supports the maxWidth. The new JMultilineLabel will not work if put into JScrollBox or similar, see class description. If you find any bugs related to this, please assign them to me.
html in UI message