<table><tr><td style="">cblack created this revision.<br />cblack added reviewers: Kirigami, VDG.<br />Herald added a project: Kirigami.<br />Herald added a subscriber: plasma-devel.<br />filipf added a comment.<br />cblack updated this revision to Diff 82775.<br />cblack edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-a66w6276jlqzbl6/">(Show Details)</a><br />cblack updated this revision to Diff 82776.<br />cblack updated this revision to Diff 82777.<br />cblack updated this revision to Diff 82778.<br />cblack updated this revision to Diff 82780.<br />cblack requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D29694">View Revision</a></tr></table><br /><div><div><p>Cool stuff. The commit message needs to be expanded to explain where and how this will be used, and also needs to list key implentation feats (initials, colors).</p>
<p>I believe we use avatars in launcher menus, the user menu, and lock, login, logout (3L) screens. The 3L have no action involved with the avatar so there's no need to buttonify it in any way for that use case. Visually they're fine as is. Launcher menus and the kcm are obviously different. But the main question is actually what do we intend to do with not using Kirigami on the desktop? That makes this component only usable in kcms?</p></div></div><br />cblack added a comment.<br /><div><div><p>Add documentation; fix bugs.</p></div></div><br />cblack added a comment.<br /><div><div><p>EOF newline</p></div></div><br />cblack added a comment.<br /><div><div><p>EOF newline for the QML file</p></div></div><br />cblack added a comment.<br /><div><div><p>Code hygiene</p></div></div><br />cblack added a comment.<br /><div><div><p>Comment the code.</p></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">{F8317303} (2x scale)<br />
<br />
```<br />
Row {<br />
spacing: Kirigami.Units.smallSpacing<br />
Kirigami.Avatar {<br />
name: "Carson Black"<br />
}<br />
Kirigami.Avatar {<br />
name: "Marco Martin"<br />
}<br />
Kirigami.Avatar {<br />
name: "Carl Schwan"<br />
}<br />
Kirigami.Avatar {<br />
name: "Kai Uwe Broulik"<br />
}<br />
Kirigami.Avatar {<br />
name: "Nate Graham"<br />
}<br />
Kirigami.Avatar {<br />
name: "Roman Gilg"<br />
initialsMode: Kirigami.Avatar.InitialsMode.UseIcon<br />
}<br />
Kirigami.Avatar {<br />
name: "David Redondo"<br />
source: "/home/jpontaoski/Pictures/Barista/barista.svg.png"<br />
}<br />
}<br />
```</span></div></div></div><br /><div><strong>REVISION SUMMARY</strong><div><p>An Avatar component displays an avatar.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p><a href="https://phabricator.kde.org/F8317303" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F8317303: ksnip_20200513-134838.png</a> (2x scale)</p>
<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">Row {
spacing: Kirigami.Units.smallSpacing
Kirigami.Avatar {
name: "Carson Black"
}
Kirigami.Avatar {
name: "Marco Martin"
}
Kirigami.Avatar {
name: "Carl Schwan"
}
Kirigami.Avatar {
name: "Kai Uwe Broulik"
}
Kirigami.Avatar {
name: "Nate Graham"
}
Kirigami.Avatar {
name: "Roman Gilg"
initialsMode: Kirigami.Avatar.InitialsMode.UseIcon
}
Kirigami.Avatar {
name: "David Redondo"
source: "/home/jpontaoski/Pictures/Barista/barista.svg.png"
}
}</pre></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R169 Kirigami</div></div></div><br /><div><strong>BRANCH</strong><div><div>cblack/avatar (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29694">https://phabricator.kde.org/D29694</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/CMakeLists.txt<br />
src/avatar.cpp<br />
src/avatar.h<br />
src/controls/Avatar.qml<br />
src/kirigamiplugin.cpp</div></div></div><br /><div><strong>To: </strong>cblack, Kirigami, VDG<br /><strong>Cc: </strong>filipf, plasma-devel, fbampaloukas, GB_2, domson, dkardarakos, ngraham, apol, ahiemstra, davidedmundson, mart<br /></div>