changed cursor for selectable text on the about dialog to IBeamCursor... #1201

• removed an an unused XML-tag

textFormat is not an unused tag ! To reduce the surprise factor, always prefer an explicit text format (PlainText, RichText) to AutoText, especially when user-provided input is involved (not the case here but still...). This prevents bugs such as this: http://vigilance.fr/vulnerability/KDE-certificate-spoofing-via-KSSL-and-Rekonq-11028

Well the default displayed in Qt Designer is Qt::AutoText, so I'm sure this is default and need's not to be added then? Default values in the Designer are non bold and changed values / settings are bold in the right attribute pane.

A I read your link ... well I guess there are more places, where there is the default of Qt:AutoText used, is that a problem then? I even think it is no problem in the about dialog, no?

Check the diff. It was RichText, not AutoText. Just keep it there, it's not hurting anything.

Right and as the text is in no way formatted (only line-feeds), it's save to use the default I think. If that gets plainText or RichText does not matter in that place. Well I can revert, no problem, but do you really think this is an security issue here? I only changed to save us a few lines.

Please just change it back. Saving lines in the xml files is not a goal, there's just no need to change it in the first place.

Being explicit (and reducing potential uncertain behavior) is a goal. If you encounter AutoText in other (potentially risky) places, feel free to change it.

"it's save to use the default I think" are very dangerous words in these times :)

Convinced :), will revert ...

Updated...