ngraham added a comment.

  There's nothing inherently wrong with using a QQC2 `GridLayout`, BTW. We 
typically port these to use a Kirigami `FormLayout` when it's a settings UI 
with a lot of controls, but this doesn't really fall under that category. For 
this, the layout probably doesn't need any changes, but using a raw `Text` 
instead of a `Label` or a `Heading` is kind of a code smell.

REPOSITORY
  R123 SDDM Configuration Panel (KCM)

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

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

Reply via email to