D6188: [Desktop] Pad cellWidth/cellHeight with the extra space

Chris Holland noreply at phabricator.kde.org
Mon Jun 12 20:29:12 UTC 2017


Zren added inline comments.

INLINE COMMENTS

> anthonyfieroni wrote in FolderView.qml:538
> Why it used (6 * units.smallSpacing) it makes more space at bottom? It should be same as iconwidth calculation, about me. I see you not change it, but it should.

Fairly certain was `smallSpacing * 2` + `iconSize` + `smallSpacing * 2` + `defaultFont.height * lines` + `smallSpacing * 2`

Right now:

- `frame` has `y: root.useListViewMode ? 0 : units.smallSpacing`
- `frame` has `height: (icon.height + (2 * units.smallSpacing) + (label.lineCount * theme.mSize(theme.defaultFont).height) + (3 * units.smallSpacing));`
- `icon` has `topMargin: (2 * units.smallSpacing)`
- `label` has `anchors.topMargin: units.smallSpacing`

So it's actually:

  smallSpacing | frame | smallSpacing*2 | icon | smallSpacing | text | smallSpacing*2

REPOSITORY
  R119 Plasma Desktop

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

To: Zren, hein
Cc: anthonyfieroni, broulik, plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170612/4b372fce/attachment-0001.html>


More information about the Plasma-devel mailing list