Review Request 127661: Phabricator T1661: Re-positioning axis label.

Alexander Semke alexander.semke at web.de
Sun Apr 24 09:04:57 UTC 2016


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




src/backend/worksheet/plots/cartesian/Axis.cpp (line 2007)
<https://git.reviewboard.kde.org/r/127661/#comment64401>

    remove whitespaces.



src/kdefrontend/ui/labelwidget.ui (line 314)
<https://git.reviewboard.kde.org/r/127661/#comment64402>

    use lOffsetX



src/kdefrontend/ui/labelwidget.ui (line 324)
<https://git.reviewboard.kde.org/r/127661/#comment64403>

    use sbOffsetX


- Alexander Semke


On April 16, 2016, 10:34 vorm., Anu Mittal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127661/
> -----------------------------------------------------------
> 
> (Updated April 16, 2016, 10:34 vorm.)
> 
> 
> Review request for KDE Edu, Alexander Semke and Garvit Khatri.
> 
> 
> Repository: labplot
> 
> 
> Description
> -------
> 
> These are the changes I have made till now, have implemented the idea of changing offset to offsetX and offsetY.But according to the discussion which I had with asemke on phabricator task I will now remove the limits of offset and rename it to 'x' and 'y' and then finally make it dragable with the click of mouse.
> 
> 
> Diffs
> -----
> 
>   src/backend/worksheet/plots/cartesian/Axis.h 03932c6 
>   src/backend/worksheet/plots/cartesian/Axis.cpp b56d9c5 
>   src/backend/worksheet/plots/cartesian/AxisPrivate.h cc0a659 
>   src/backend/worksheet/plots/cartesian/CartesianPlot.cpp bdfae24 
>   src/kdefrontend/ui/labelwidget.ui 06b058f 
>   src/kdefrontend/widgets/LabelWidget.h 72bcf09 
>   src/kdefrontend/widgets/LabelWidget.cpp 4ac385c 
> 
> Diff: https://git.reviewboard.kde.org/r/127661/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Anu Mittal
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20160424/7b6f8f13/attachment.html>


More information about the kde-edu mailing list