Cosmetic Issues 3.8 - WYSIWYG, Round corners, FCKeditor not showing

Hi Everyone,

Got couple of cosmetic issues I’m trying to solve with RT 3.8.1

  1. Entering text in the WYSIWYG editor looks fine, I make the font big,
    change the color to Red - but when you view the ticket history it just
    shows as all the same size/font,

    is gone, Red color is gone. (it is
    there in the Download (untitled) / with headers text/html, so RT did
    save it)

  2. Message box height doesn’t alter when I change it in preferences, or
    in RT_SiteConfig.pm, I suspect this issue is related to the FCKeditor,
    I’ve tried adjusting fckeditor.js FCKeditor.MinHeight = 400 ; as well
    but it doesn’t seem to have the desired effect.

  3. Using IE7 the rounded corners are missing, only working in Firefox.

Not really show stopper issues, but most people use IE here & with the
FCK window expanded the message box remaining is pretty tiny.

Thanks,
Scott

  1. Entering text in the WYSIWYG editor looks fine, I make the font
    big,
    change the color to Red - but when you view the ticket history it just
    shows as all the same size/font,

    is gone, Red color is gone.
    (it is
    there in the Download (untitled) / with headers text/html, so RT did
    save it)

There are fixes for this in 3.8-trunk which will be in 3.8.2

  1. Message box height doesn’t alter when I change it in preferences,
    or
    in RT_SiteConfig.pm, I suspect this issue is related to the FCKeditor,
    I’ve tried adjusting fckeditor.js FCKeditor.MinHeight = 400 ; as well
    but it doesn’t seem to have the desired effect.

There is a new config option for this which will be in 3.8.2

  1. Using IE7 the rounded corners are missing, only working in Firefox.

I believe this was to avoid using javascript to achieve rounded corners
(firefox and safari have css support that make it easy). I’d be happy
to see a patch for it.

-kevin

Kevin Falcone wrote:> On Nov 12, 2008, at 4:00 PM, Scott Golby wrote:

  1. Entering text in the WYSIWYG editor looks fine, I make the font
    big,
    change the color to Red - but when you view the ticket history it just
    shows as all the same size/font,

    is gone, Red color is gone.
    (it is
    there in the Download (untitled) / with headers text/html, so RT did
    save it)

There are fixes for this in 3.8-trunk which will be in 3.8.2
Hi Kevin,
I’ve applied your patch from the 3.8 trunk
[http://svn.bestpractical.com/cgi-bin/index.cgi/bps/revision/?rev=16201#diff__rt_3.8_trunk_share_html_Elements_ScrubHTML]
, but found it doesn’t include H1 → H6 headings.

I enclose a diff to add H1 → H6.

Best regards,
Justin

diff__rt_3.8_trunk_share_html_Elements_ScrubHTML.diff (551 Bytes)

Kevin Falcone wrote:

  1. Entering text in the WYSIWYG editor looks fine, I make the font
    big,
    change the color to Red - but when you view the ticket history it
    just
    shows as all the same size/font,

    is gone, Red color is gone.
    (it is
    there in the Download (untitled) / with headers text/html, so RT
    did
    save it)

There are fixes for this in 3.8-trunk which will be in 3.8.2
Hi Kevin,
I’ve applied your patch from the 3.8 trunk [http://svn.bestpractical.com/cgi-bin/index.cgi/bps/revision/?rev=16201#diff__rt_3.8_trunk_share_html_Elements_ScrubHTML
]
, but found it doesn’t include H1 → H6 headings.

I enclose a diff to add H1 → H6.

Hi Justin

Can you tell me how you were generating the H* from the WYSIWYG so I
can confirm
before committing?

Thanks

-kevin

Kevin Falcone wrote:> On Nov 26, 2008, at 3:04 AM, justin@brighton.ac.uk wrote:

Kevin Falcone wrote:

On Nov 12, 2008, at 4:00 PM, Scott Golby wrote:

  1. Entering text in the WYSIWYG editor looks fine, I make the font
    big,
    change the color to Red - but when you view the ticket history it
    just
    shows as all the same size/font,

    is gone, Red color is gone.
    (it is
    there in the Download (untitled) / with headers text/html, so RT
    did
    save it)

There are fixes for this in 3.8-trunk which will be in 3.8.2

Hi Kevin,
I’ve applied your patch from the 3.8 trunk [http://svn.bestpractical.com/cgi-bin/index.cgi/bps/revision/?rev=16201#diff__rt_3.8_trunk_share_html_Elements_ScrubHTML
]
, but found it doesn’t include H1 → H6 headings.

I enclose a diff to add H1 → H6.

Hi Justin

Can you tell me how you were generating the H* from the WYSIWYG so I
can confirm
before committing?

Thanks

-kevin

Hi Kevin,
Thanks for the reply.
We’re using the ‘Format’ drop down box in FCKeditor. Included in the
options are ‘Heading 1’ through to ‘Heading 6’.

Best regards,
Justin
University of Brighton.