-----------------------------------------------------------
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
> 
>

_______________________________________________
Marble-devel mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/marble-devel

Reply via email to