KNotificationItem specification - first draft

[Marco Martin]

On Tuesday 08 September 2009, Aurélien Gâteau wrote:
> Aaron J. Seigo wrote:
> >>>> or:
> >>>> State that if an item wishes to use markup, then it must enclose the
> >>>> whole text in a <markup> tag.
> >>>
> >>> that would work as well. it's a small amount of overhead for not much
> >>> pain. it could also be that we look for the "<pre>" tag instead, and
> >>> make markup the default.
> >>
> >> Doing it this way is not good. It can fail in the (admittedly rare) case
> >> where the plain text contains the closing tag.
> >
> > how is that different from using <markup>? this just says that rich text
> > is the default, and if you want plain text to use <pre> around your text.
>
> Imagine an application which does not markup, but shows text which is
> not known at compile time. From the app point of view, it uses some a
> "template like this:
>
> <pre>Static text\n%1\nMore static text</pre>
>
> Then in the (admittedly rare) case where %1 is "Foo</pre>", your text is
> cut. Unless of course you do not check for the closing tag, which
> (thinking aloud) you don't really need to... but it feels a bit more
> hackish than having to escape markup.
>
> > or maybe a less hackish way to go about it all would be to add a bool
> > member to the ToolTip structure that says whether or not this is markup
> > or not.
>
> Yes, it's probably simpler.

agree

> > in practice, i'm not sure how much this matters one way or the other.
> > we've had rich text tooltips for a while now and there hasn't been any
> > incidents with "but i wanted plain text only! the tooltip unexpectedly
> > looks like crap now!"
>
> Things can go wrong when an app uses dynamic text and the content of the
> dynamic text contents markup which the app did not escape because it
> thought everything was plain-text. By not making markup the default, you
> avoid this.

well, if the protocol mandates the text has markup the app has to support it 
no?
however i'm fine with adding a new paramer to the tooltip struct to define 
this

> Aurelien
> _______________________________________________
> xdg mailing list
> xdg at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/xdg


-- 
Marco Martin