Re: New repo in kdereview: KWeather

2022-11-24 Thread Devin
> There's potentially overlapping text. I have fixed text wrapping in the weather delegates so it should wrap properly now. > Here settingsModel.forecastStyle is not i18n'ed I have fixed it now (it converts to i18n'd dedicated strings in the QML) On Mon, Nov 14, 2022 at 5:10 PM Albert Astals Ci

Re: New repo in kdereview: KWeather

2022-11-24 Thread Devin
> The main page has no scrollbars, so it's not obvious that the view is > scrollable, especially because with various window sizes, nothing looks > visibly cut off on the bottom to suggest scrollability, which is the typical > metaphor on mobile: https://i.imgur.com/mN7AD09.jpg I have added scr

Re: New repo in kdereview: KRecorder

2022-11-24 Thread Devin
> I can still see this: https://i.imgur.com/MrrwyAo.jpg No matter how I try to resize the window, I can't seem to reproduce the issue... weird > When I look at other platforms, what I see is that it's common and > traditional for the *record* button to be a red circle, but the stop button > is

Re: Tokodon in KDE Review

2022-11-24 Thread Tobias Fella
Hi, - Reuse lint is almost happy, would be nice to complete that and have it in the CI - As reported in 461846, some icons aren't visible - The "Global" view doesn't open for me and complains qrc:/content/ui/main.qml:162: ReferenceError: TimelineType is not defined Other than that it work

Tokodon in KDE Review

2022-11-24 Thread Carl Schwan
Hello everyone, I would like to put Tokodon in KDE Review. Tokodon is mastodon client, I have been working for some time already. I already did some unstable release and I would like to soon release the first stable release. The source code is available here: https://invent.kde.org/network/tokodo