wsdfhjxc added a comment.

  I can no longer reproduce the issue with incorrect visibility state in 
checkboxes. It seems to be working correctly after last patch.
  
  Also, I've added some minor styling suggestions, so it looks a little bit 
more polished in my opinion.

INLINE COMMENTS

> ConfigEntries.qml:201
> +                            QtLayouts.Layout.preferredWidth: 1
> +                            QtLayouts.Layout.fillHeight: true
> +                        }

height: measureButton.height

> ConfigEntries.qml:319
> +
> +            Rectangle {
> +                color: "black"

Rectangle {
      SystemPalette {
          id: sysPalette
          colorGroup: SystemPalette.Active
      }
                  
      color: sysPalette.mid
      opacity: 0.85
      anchors.fill: parent
  }

> ConfigEntries.qml:328
> +                anchors.margins: Math.round(units.gridUnit * 0.2)
> +                icon: "trash-empty"
>              }

icon: "albumfolder-user-trash"

REPOSITORY
  R120 Plasma Workspace

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

To: Pitel, #plasma, #vdg
Cc: zzag, davidedmundson, anthonyfieroni, wsdfhjxc, ngraham, plasma-devel, 
ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

Reply via email to