[Marble-devel] Review Request 122944: Marble: EditPlacemarkDialog - placemark description fix.
Calin Cruceru
crucerucalincristian at gmail.com
Fri Mar 20 10:27:06 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122944/#review77814
-----------------------------------------------------------
Let's change it to back "Formatted text" as this is a pretty standard way to refer to HTML embellished text. Other than that, it seems ok to me to have it checked by default.
- Calin Cruceru
On March 14, 2015, 3:42 p.m., Constantin Mihalache wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122944/
> -----------------------------------------------------------
>
> (Updated March 14, 2015, 3:42 p.m.)
>
>
> Review request for Marble and Calin Cruceru.
>
>
> Repository: marble
>
>
> Description
> -------
>
> #1 EditPlacemarkDialog
> - set "Formatted text" as checked by default. This seems more intuitive for a standard user and you can think of unformatted text as a "View HTML" option.
>
> `This justifies the HTML that appeares when viewing unformatted text.`
>
>
> Diffs
> -----
>
> src/lib/marble/EditPlacemarkDialog.cpp e1d9825
> src/lib/marble/EditPlacemarkDialog.ui 2d09c6e
>
> Diff: https://git.reviewboard.kde.org/r/122944/diff/
>
>
> Testing
> -------
>
> Works for me.
>
>
> Thanks,
>
> Constantin Mihalache
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20150320/1c170d24/attachment.html>
More information about the Marble-devel
mailing list