> Ok, now that i updated kirigami-addons, the Settings dialog shows, but it's > uncloseable, any idea why that would be?
That's very strange, it should be showing up as a regular window, does pressing the close button on the window decoration not work? Thanks, Devin On Tue, Nov 29, 2022 at 6:23 PM Albert Astals Cid <aa...@kde.org> wrote: > > El dilluns, 28 de novembre de 2022, a les 17:15:57 (CET), Albert Astals Cid va > escriure: > > El divendres, 25 de novembre de 2022, a les 4:46:40 (CET), Devin va > escriure: > > > > 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) > > > > FWIW i can't open settings anymore > > > > qrc:/qml/settings/SettingsWindow.qml:56:13: Type SettingsComponent > > unavailable qrc:/qml/settings/SettingsComponent.qml:61:17: Cannot assign to > > non-existent property "onActivated" qrc:/qml/main.qml:136: TypeError: > > Cannot call method 'close' of null > > Ok, now that i updated kirigami-addons, the Settings dialog shows, but it's > uncloseable, any idea why that would be? > > Cheers, > Albert > > > > > Cheers, > > Albert > > > > > On Mon, Nov 14, 2022 at 5:10 PM Albert Astals Cid <aa...@kde.org> wrote: > > > > El dimecres, 9 de novembre de 2022, a les 23:00:13 (CET), Devin va > escriure: > > > > > Hi everyone, > > > > > > > > > > I would like to put kweather through kdereview: > > > > > > > > > > https://invent.kde.org/plasma-mobile/kweather > > > > > > > > > > KWeather is an application that can give simple weather information > > > > > for different weather locations. Please note that KWeatherCore (the > > > > > library the app depends on) has already passed kdereview. > > > > > > > > There's potentially overlapping text. > > > > > > > > See https://i.imgur.com/BrOgi3A.png > > > > > > > > The settings show untranslateable text, i.e. > > > > > > > > MobileForm.FormComboBoxDelegate { > > > > > > > > id: forecastStyleDropdown > > > > text: i18n("Forecast Style") > > > > currentValue: settingsModel.forecastStyle > > > > > > > > Here settingsModel.forecastStyle is not i18n'ed > > > > > > > > Not KWeather's fault but i just realized that Kirigami's about page > > > > doesn't > > > > properly credit translators :/ > > > > > > > > Cheers, > > > > > > > > Albert > > > > > > > > > Thanks, > > > > > Devin > > > >