[Marble-devel] Review Request 123122: Formatted text widget.

Constantin Mihalache mihalache.c94 at gmail.com
Wed Apr 1 23:40:58 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123122/
-----------------------------------------------------------

(Updated April 1, 2015, 11:40 p.m.)


Review request for Marble, Calin Cruceru and Torsten Rahn.


Changes
-------

add explicit keyword to FormattedTextWidget constructor. change getText() to text().


Repository: marble


Description
-------

#1 Widget implementation:
I've respected the structured given by EditPlacemarkDialog, which had formatted description.
I've first added this functionality to Polygon, Polyline and GroundOverlay editing.
I've also replaced the code in EditPlacemarkDialog with the widget.
#2 Solved a visual bug in EditPolygonDialog and EditPolylineDialog:
The node table header didn't stretch with the window.


Diffs (updated)
-----

  src/lib/marble/CMakeLists.txt 6c65015 
  src/lib/marble/EditPlacemarkDialog.h 015173c 
  src/lib/marble/EditPlacemarkDialog.cpp 3d0d61f 
  src/lib/marble/EditPlacemarkDialog.ui e17bf14 
  src/lib/marble/FormattedTextWidget.h PRE-CREATION 
  src/lib/marble/FormattedTextWidget.cpp PRE-CREATION 
  src/lib/marble/FormattedTextWidget.ui PRE-CREATION 
  src/plugins/render/annotate/EditGroundOverlayDialog.cpp df7d6d2 
  src/plugins/render/annotate/EditGroundOverlayDialog.ui 1151b35 
  src/plugins/render/annotate/EditPolygonDialog.cpp c647e6b 
  src/plugins/render/annotate/EditPolygonDialog.ui 563ca2d 
  src/plugins/render/annotate/EditPolylineDialog.cpp a99a3e9 
  src/plugins/render/annotate/EditPolylineDialog.ui 3421a72 

Diff: https://git.reviewboard.kde.org/r/123122/diff/


Testing
-------

It works.


Thanks,

Constantin Mihalache

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20150401/88017a0d/attachment-0001.html>


More information about the Marble-devel mailing list