Review Request 119467: Implement the main UI in QML (Kadixt patches 1/3)

Ovidiu-Florin Bogdan ovidiu.b13 at gmail.com
Sun Aug 17 07:59:39 UTC 2014



> On July 25, 2014, 3:21 p.m., David Edmundson wrote:
> > src/qml/Main.qml, line 101
> > <https://git.reviewboard.kde.org/r/119467/diff/1/?file=292737#file292737line101>
> >
> >     i18n
> 
> Ovidiu-Florin Bogdan wrote:
>     File does not exist anymore in latest commit in http://quickgit.kde.org/?p=ktp-call-ui.git&a=shortlog&h=25f1f05c9f6eb7609170f2094f7ddffa10c575da
> 
> Diane Trout wrote:
>     You're probably on the wrong branch. e.g. the kaditx_qml branch has it here http://quickgit.kde.org/?p=ktp-call-ui.git&a=blob&h=90a4e616b3875ba5301143188940a38eb2b0f9d3&hb=caca2696c2de09aa08241bb1215a63cff8eb6bce&f=src%2Fqml%2FMain.qml
> 
> Ovidiu-Florin Bogdan wrote:
>     Files in stc/qml are not added in the CMakeLists.txt project, The IDE did not see them. They exist in the filesystem.
>     
>     Anyhow, fixed in a45fa8467f77776946b68484d96751ce5d869a27

I couldn't find the commit using the quickgit.kde.org interface, so here's the link: http://quickgit.kde.org/?p=ktp-call-ui.git&a=commit&h=a45fa8467f77776946b68484d96751ce5d869a27


- Ovidiu-Florin


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


On Aug. 2, 2014, 11:28 a.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119467/
> -----------------------------------------------------------
> 
> (Updated Aug. 2, 2014, 11:28 a.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Repository: ktp-call-ui
> 
> 
> Description
> -------
> 
> Implement the main UI in QML (Kadixt patches 1/3)
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 8c39e7c 
>   src/CMakeLists.txt 250aeb5 
>   src/call-window.h 07fd01e 
>   src/call-window.cpp c38112d 
>   src/call-window.ui 32c7dad 
>   src/callwindowui.rc 6c390b9 
>   src/dtmf-handler.h 91960dc 
>   src/dtmf-handler.cpp d8d7970 
>   src/dtmf-qml.h PRE-CREATION 
>   src/dtmf-qml.cpp PRE-CREATION 
>   src/dtmf-widget.h 9e1bc73 
>   src/dtmf-widget.cpp f3436b2 
>   src/dtmf-widget.ui 67f60b9 
>   src/qml-interface.h PRE-CREATION 
>   src/qml-interface.cpp PRE-CREATION 
>   src/qml/Main.qml PRE-CREATION 
>   src/qml/core/Button.qml PRE-CREATION 
>   src/qml/core/Dtmf.qml PRE-CREATION 
>   src/qml/core/DtmfButton.qml PRE-CREATION 
>   src/qml/core/Label.qml PRE-CREATION 
>   src/qml/core/Separator.qml PRE-CREATION 
>   src/qml/core/ToggleButton.qml PRE-CREATION 
>   src/qml/core/Toolbar.qml PRE-CREATION 
>   src/qml/core/Tooltip.qml PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/119467/diff/
> 
> 
> Testing
> -------
> 
> Rebased, patched and cherry-picked until I was red in the face.
> 
> Compiles, not tested beyond that.
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20140817/e1d791c8/attachment.html>


More information about the KDE-Telepathy mailing list