D26221: Use QQC2 Dialog

Dan Leinir Turthra Jensen noreply at phabricator.kde.org
Thu Dec 26 14:22:12 GMT 2019


leinir added a comment.


  That's looking pretty good, really :) Also code reduction is certainly good ;)

INLINE COMMENTS

> Dialog.qml:136
>  
> -            anchors {
> -                left: parent.left;
> -                leftMargin: 8;
> -                right: parent.right;
> -                bottom: parent.bottom;
> -                bottomMargin: 8;
> +            QQC2.BusyIndicator {
> +                running: base.enabled

Thinking this wants to be centered... or if that means things end up looking lop-sided, perhaps have it right-aligned? As it stands, it looks kind of... very heavy on the left hand side.

REPOSITORY
  R8 Calligra

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

To: ognarb, #calligra:_3.0, leinir
Cc: Calligra-Devel-list, davidllewellynjones, dcaliste, ognarb, cochise, vandenoever
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20191226/4886e550/attachment.htm>


More information about the calligra-devel mailing list