Differential D11316 Diff 29626 src/controls/CardsLayout.qml

Changeset View

Standalone View

View Options

src/controls/CardsLayout.qml

  • This file was added.
1​/*
2​ * Copyright 2018 Marco Martin <mart@kde.org>
3​ *
4​ * This program is free software; you can redistribute it and/or modify
5​ * it under the terms of the GNU Library General Public License as
6​ * published by the Free Software Foundation; either version 2 or
7​ * (at your option) any later version.
8​ *
9​ * This program is distributed in the hope that it will be useful,
10​ * but WITHOUT ANY WARRANTY; without even the implied warranty of
11​ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12​ * GNU Library General Public License for more details
13​ *
14​ * You should have received a copy of the GNU Library General Public
15​ * License along with this program; if not, write to the
16​ * Free Software Foundation, Inc.,
17​ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
18​ */
19​
20​import QtQuick 2.6
21​import QtQuick.Controls 2.0 as Controls
22​import QtQuick.Layouts 1.2
23​import org.kde.kirigami 2.4 as Kirigami
24​
25​/**
26​ * A GridLayout optimized for showing one or two columns of cards,
27​ * depending on the available space.
28​ * It Should be used when the cards are not instantiated by a model or by a
29​ * model which has always very few items (In the case of a big model
30​ * CardsListView or CardsGridview should be used instead).
31​ * They are presented as a grid of two columns which will remain
32​ * centered if the application is really wide, or become a single
33​ * column if there is not enough space for two columns,
34​ * such as a mobile phone screen.
35​ * A CardsLayout should always be contained within a ColumnLayout.
36​ * @inherits GridLayout
davidedmundsonUnsubmitted
Not Done

I can see you want it in a layout because you're setting the relevant attached properties, but why explicitly Column?

davidedmundson: I can see you want it in a layout because you're setting the relevant attached properties, but…
martAuthorUnsubmitted
Not Done

It's not mandatory but the one i think it makes more sense, so i want to put it as a recommendation.
kirigami pages always scroll only vertically, so makes sense to recommend a vertical development of the final page layout.

mart: It's not mandatory but the one i think it makes more sense, so i want to put it as a…
37​ * @since 2.4
38​ */
39​GridLayout {
40​ /**
41​ * maximumColumnWidth: int
42​ * The maximum width the columns may have. the cards will never
43​ * get wider than this size, when the GridLayout is wider than
44​ * maximumColumnWidth, it will switch from one to two columns.
45​ * If the default needs to be overridden for some reason,
46​ * it is advised to express this unit as a multiple
47​ * of Kirigami.Units.gridUnit
48​ */
49​ property int maximumColumnWidth: Kirigami.Units.gridUnit * 20
50​
51​ columns: width > maximumColumnWidth ? 2 : 1
52​ rowSpacing: Kirigami.Units.largeSpacing * 2
53​ columnSpacing: Kirigami.Units.largeSpacing * 2
54​
55​ Layout.preferredWidth: maximumColumnWidth * 2 + (columns > 1 ? Kirigami.Units.largeSpacing * 2 : 0)
56​ Layout.maximumWidth: Layout.preferredWidth
davidedmundsonUnsubmitted
Done

shouldn't this include the spacing? Otherwise it's always ever so slightly smaller than my maxSize

davidedmundson: shouldn't this include the spacing? Otherwise it's always ever so slightly smaller than my…
57​ Layout.alignment: Qt.AlignHCenter
58​
59​ Component.onCompleted: childrenChanged()
60​ onChildrenChanged: {
61​ for (var i = 0; i < children.length; ++i) {
62​ children[i].Layout.fillHeight = true;
63​ }
64​ }
65​}