KTrip in kdereview

[Nicolas Fella]

On Dienstag, 29. Oktober 2019 23:25:44 CET Albert Astals Cid wrote:
> El diumenge, 27 d’octubre de 2019, a les 15:29:10 CET, Nicolas Fella va 
escriure:
> You're not calling
>   KLocalizedString::setApplicationDomain
> as far as i can see so translations won't work.
Fixed.

> It would be good for it to use feature_summary so instead of a hard to read
> error http://paste.debian.net/1111810/ one would get the nice summary with
> "you're missing this required package, find it here"
I'm using feature_summary. What should I do differently?
> Is running on the desktop supposed to work? http://paste.debian.net/1111811/
Yes. I've fixed most of the warnings about Android stuff. There's still a few 
left, needs a closer look from me.
> The time picker looks megaweird on the desktop
> https://i.imgur.com/ksBMhzj.png
Agreed. I didn't do that myself, it's a copy and paste from Calindori. We need 
to sit down with VDG and design something and then implement is in a proper 
lib.
> What's the 3rd box here supposed to be? https://i.imgur.com/PQMvsgP.png Is
> it "wait there for a bit"?
yes, waiting or transfer. I fixed the appearance a bit, but it's still not the 
most beautiful design
> Cheers,
>   Albert
> 
Thanks,

Nico