MauiKit and Index review

[Camilo Higuita Rodriguez]

I will make the corrections to mauikit. Not sire how is the translation
catalog set?
What is the best way to set the date format to not force it?

Thanks for the feedback.

On Mon, 28 Sep 2020 at 5:42 PM Albert Astals Cid <aacid at kde.org> wrote:

> El dimarts, 22 de setembre de 2020, a les 23:05:10 CEST, Camilo Higuita
> Rodriguez va escriure:
>
> > Hi,
>
> > For the next stable release, we would like to go through KDE review
> first.
>
> >
>
> > To start I want to submit MauiKit and Index for review, and later on the
>
> > other apps.
>
> > https://invent.kde.org/maui/mauikit
>
> > https://invent.kde.org/maui/index
>
> >
>
> > The changes we have made to get to this review are all in the development
>
> > branch.
>
> >
>
> > I will be available to perform any needed fixes and answer any questions
>
> > since I'm the main developer and maintainer.
>
>
>
> mauikit
>
>
>
> ./src/controls/private/AboutDialog.qml:151:
> label1.text: "Powered by"
>
> ./src/controls/FileBrowser.qml:164:                     title:  "Removing
> %1 files".arg(urls.length)
>
> ./src/controls/FileBrowser.qml:119:             placeholderText:
> String("Filter %1 files").arg(control.currentFMList ?
> control.currentFMList.count : 0)
>
>         needs i18n?
>
>
>
> ./src/controls/Dialog.qml:214:                text: qsTr("Cancel")
>
> ./src/controls/Dialog.qml:259:                text: qsTr("Accept")
>
>         should be i18n instead of qsTr?
>
>
>
> ./src/controls/ListBrowserDelegate.qml:98:        label4.text:
> model.modified ? Maui.FM.formatDate(model.modified, "MM/dd/yyyy") : ""
>
>         Forcing the date format seems quite bad
>
>
>
>
>
> You don't have a Messages.sh in mauikit. If this is a library, shouldn't
> it be using i18nd instead of i18n? How is the translation catalog loaded
> otherwise?
>
>
>
>
>
> index-fm
>
> You have a Messages.sh but the translation catalog is not set anywhere
> that i can see?
>
>
>
> Cheers,
>
>   Albert
>
>
>
>
>
> >
>
> > Greetings,
>
> > Camilo Higuita
>
> >
>
> >
>
>
>
>
>
>
>
>
>
>

-- 
*Aviso legal:* El contenido de este mensaje y los archivos adjuntos son 
confidenciales y de uso exclusivo de la Universidad Nacional de Colombia. 
Se encuentran dirigidos sólo para el uso del destinatario al cual van 
enviados. La reproducción, lectura y/o copia se encuentran prohibidas a 
cualquier persona diferente a este y puede ser ilegal. Si usted lo ha 
recibido por error, infórmenos y elimínelo de su correo. Los Datos 
Personales serán tratados conforme a la Ley 1581 de 2012 y a nuestra 
Política de Datos Personales que podrá consultar en la página web 
www.unal.edu.co <http://www.unal.edu.co/>.* *Las opiniones, informaciones, 
conclusiones y cualquier otro tipo de dato contenido en este correo 
electrónico, no relacionados con la actividad de la Universidad Nacional de 
Colombia, se entenderá como personales y de ninguna manera son avaladas por 
la Universidad.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20200929/470d53fb/attachment.htm>