nicolasfella marked an inline comment as done. nicolasfella added inline comments.
INLINE COMMENTS > broulik wrote in TimeDialog.qml:15 > Generally conversions between C++ timedate and JavaScript `Date` is bad. > There's no way to represent just a time with no date and timezone associated > with it in JavaScript. > While it's ugly, I'd suggest we return a bunch of `int`. > Also, I think we should have the selected time as properties and an > `accepted`/rejected signal or similar. > (Same probably goes for the date picker) If we do this should the date thing also use ints for symmentry? REPOSITORY R1047 Kirigami Addons REVISION DETAIL https://phabricator.kde.org/D29268 To: nicolasfella, davidedmundson, vkrause, broulik Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
