Differential D8548

Align app icon with app name, not back button
AbandonedPublic
Actions

Authored by ngraham on Oct 30 2017, 4:54 AM.

Details

Reviewers
apol
jensreuterberg
Group Reviewers
Discover Software Store
VDG
Summary

BUG: 384916
Align the app icon with the app's name, not the back button

Test Plan

Tested in KDE Neon:

Before, installable app:

After, installable app:

Before, installed app:

After, installed app:

Diff Detail

Repository
R134 Discover Software Store
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
ngraham created this revision.Oct 30 2017, 4:54 AM
Restricted Application added a project: Plasma. · View Herald TranscriptOct 30 2017, 4:54 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
ngraham edited the test plan for this revision. (Show Details)Oct 30 2017, 4:56 AM
ngraham added reviewers: Discover Software Store, apol.
ngraham edited projects, added Discover Software Store; removed Plasma.
ngraham removed a subscriber: plasma-devel.
apol edited reviewers, added: jensreuterberg; removed: abetts.Oct 30 2017, 1:10 PM
jensreuterberg added a comment.Oct 30 2017, 3:41 PM

I think this is pretty damn spot on and I think the revision makes sense. Awesome work Nate!

ngraham abandoned this revision.Oct 31 2017, 2:37 AM

Seems better, thanks!

Revision Contents
Changeset List

PathPackages
Mdiscover/qml/ApplicationPage.qml (33 lines)

Diff 21543

View Options

discover/qml/ApplicationPage.qml

Loading...