Review Request 124004: Add week numbers to calendar - Part 1
Martin Klapetek
martin.klapetek at gmail.com
Thu Jun 4 17:18:10 UTC 2015
> On June 4, 2015, 11:57 a.m., Kai Uwe Broulik wrote:
> > > Somehow the Repeater { model: } does not like QList<int>
> >
> > I think that's a limitation of the JavaScript engine and/or type conversion thing. QVariantList and QStringList are converted to JS Arrays but I don't think it does that for any other QList<T>.
QList<int> should still work, according to http://doc.qt.io/qt-5/qtqml-cppintegration-data.html#sequence-type-to-javascript-array
- Martin
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124004/#review81162
-----------------------------------------------------------
On June 4, 2015, 11:53 a.m., Martin Klapetek wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124004/
> -----------------------------------------------------------
>
> (Updated June 4, 2015, 11:53 a.m.)
>
>
> Review request for Plasma.
>
>
> Repository: plasma-framework
>
>
> Description
> -------
>
> One of the most requested features for Plasma5. The backend is all there*, it just needs the frontend part. Optional, defaults to off.
>
> *Somehow the Repeater { model: } does not like QList<int>, it just doesn't show anything while QStringList works just fine. Any ideas on that?
>
>
> Diffs
> -----
>
> src/declarativeimports/calendar/calendar.h ea20bf5
> src/declarativeimports/calendar/calendar.cpp a766acb
> src/declarativeimports/calendar/qml/DaysCalendar.qml 67b3a2e
> src/declarativeimports/calendar/qml/MonthView.qml 6eaa8db
>
> Diff: https://git.reviewboard.kde.org/r/124004/diff/
>
>
> Testing
> -------
>
>
> File Attachments
> ----------------
>
> Screenshot
> https://git.reviewboard.kde.org/media/uploaded/files/2015/06/04/2283789a-91e1-4a99-b341-bcaddaaeac85__calendar-weeks6.png
>
>
> Thanks,
>
> Martin Klapetek
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150604/7191be5e/attachment.html>
More information about the Plasma-devel
mailing list