GB_2 added inline comments.

INLINE COMMENTS

> advanceconfig.ui:226
> +       <item>
> +        <widget class="QPushButton" name="syncSettings">
> +         <property name="text">

It would be nice if you added the icon `view-refresh` to this button...

> advanceconfig.ui:233
> +       <item>
> +        <widget class="QPushButton" name="resetSettings">
> +         <property name="text">

...and here the icon `document-revert` or `edit-undo`.

> advanceconfig.ui:256
> +       <property name="text">
> +        <string>Settings synchronization allows you to transfer you theme 
> customization (color scheme, font, font rendering, desktop and icon theme) to 
> SDDM.</string>
>         </property>

you theme -> your theme
desktop -> Plasma

REPOSITORY
  R123 SDDM Configuration Panel (KCM)

REVISION DETAIL
  https://phabricator.kde.org/D22191

To: filipf, #plasma, ngraham, davidedmundson, #vdg
Cc: GB_2, ndavis, plasma-devel, LeGast00n, jraleigh, fbampaloukas, ragreen, 
Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, 
sebas, apol, mart

Reply via email to