[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