[bt-devel] NASB Unlock Key

[Gary Holmlund]

Jaak,

I agree that a subset of Qt's rich text is what we want. It is a good 
starting place.

For now, we could verify that only simple <a> links exist in the 
message. If so, we display as rich text and follow the link. If not, we 
show as plain text.

Gary


On 1/6/20 11:45 PM, Jaak Ristioja wrote:
> This sounded like a good idea at first, but after having read that
> document I would say no.
>
> The problem is that the Qt rich text engine supports a lot of HTML
> features which might be problematic for us, e.g. support for colors
> might cause issues like
>
>    https://github.com/bibletime/bibletime/issues/208
>
> and the src="" attribute of <img> and the background-image: <uri> CSS
> property might allow specifying non-local URIs in which case the UI
> might attempt to download these from the Internet which fails if the
> user is offline, when online allows servers to track users and would
> fail if the resource is no longer available on the server.
>
> Secondly, we would have to use a more specific version of that spec,
> e.g. https://doc.qt.io/qt-5.12/richtext-html-subset.html. As far as
> BibleTime and other Qt frontends are concerned, this means that we might
> still have to use a QWebSomething widget instead of QLabel/QTextEdit,
> because the supported HTML for QTextDocument might change between Qt
> versions even in backwards incompatible manner.
>
> I think that some smaller subset of that subset would be better. And if
> RTFHTML would conform to that (and not crash), I'd have no problems with
> using that in BibleTime.
>
> J
>
> On 07.01.20 06:32, Gary Holmlund wrote:
>> Jaak and Troy,
>>
>> I have a suggestion. Qt supports rich text (a subset of HTML) and
>> documents it here:
>>
>> https://doc.qt.io/qt-5/richtext-html-subset.html
>>
>> It is quite rich for any messages anyone  would display for unlocking.
>> Could we agree to this spec?
>>
>> Gary Holmlund
>>
>>
>> _______________________________________________
>> bt-devel mailing list
>> bt-devel at crosswire.org
>> http://www.crosswire.org/mailman/listinfo/bt-devel
>
> _______________________________________________
> bt-devel mailing list
> bt-devel at crosswire.org
> http://www.crosswire.org/mailman/listinfo/bt-devel