[kde-edu]: Review Request: Kig: use antialiasing for QPainter

[Alexander Potashev]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5444/#review7782
-----------------------------------------------------------


I think, you should not use antialiasing for the axes and the grid, because they become wider.

- Alexander


On 2010-09-24 21:03:57, Igor Poboiko wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/5444/
> -----------------------------------------------------------
> 
> (Updated 2010-09-24 21:03:57)
> 
> 
> Review request for KDE-Edu.
> 
> 
> Summary
> -------
> 
> This simple patch adds an QPainter::Antialiasing hint for Kig's painter. 
> I think smoothed scene looks much better (actually, IMHO without it scene looks a little ugly) and a bit usablier than previous one (see attached screenshots)
> 
> P.S Looks like Kig is using it's own "reinvented wheel" to paint all objects.
> Why don't Kig use Qt Graphics View framework (QGraphicsView/QGraphicsScene/etc)? What do you think about porting Kig to it? (I know that it is an off-topic for this review request, but I want to know your ideas about it)
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdeedu/kig/misc/kigpainter.cpp 1173923 
> 
> Diff: http://svn.reviewboard.kde.org/r/5444/diff
> 
> 
> Testing
> -------
> 
> Kig from KDE 4.5.1 (tested with Qt 4.7.0) compiles with this patch and works fine.
> 
> 
> Screenshots
> -----------
> 
> Without antialiasing
>   http://svn.reviewboard.kde.org/r/5444/s/509/
> With antialiasing
>   http://svn.reviewboard.kde.org/r/5444/s/510/
> 
> 
> Thanks,
> 
> Igor
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-edu/attachments/20100925/98ba033c/attachment.htm