Differential D2013

Right align the distribution logo
ClosedPublic
Actions

Authored by ltoscano on Jun 26 2016, 3:35 PM.

Details

Reviewers

davidedmundson
jriddell
mart

Commits

R102:36341ea6c287: Right align the distribution logo
R102:68eee423a6b6: Right align the distribution logo

Summary

The other elements of the list are all right-aligned.

Diff Detail

Repository

R102 KInfoCenter

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

ltoscano updated this revision to Diff 4743.Jun 26 2016, 3:35 PM

ltoscano retitled this revision from to Right align the distribution logo.

ltoscano updated this object.

Restricted Application added a project: Plasma. · View Herald TranscriptJun 26 2016, 3:35 PM

Restricted Application added a subscriber: plasma-devel. · View Herald Transcript

... or it could be centered, but left aligned looks really strange.

If accepted, would it be still on time for Plasma 5.7?

mart accepted this revision.Jun 28 2016, 8:15 AM

mart edited edge metadata.

This revision is now accepted and ready to land.Jun 28 2016, 8:15 AM

Closed by commit R102:68eee423a6b6: Right align the distribution logo (authored by ltoscano). · Explain WhyJun 28 2016, 8:32 AM

This revision was automatically updated to reflect the committed changes.

Ouch... arcanist didn't do what I thought it should have done, and I ended up applying it to master instead of Plasma/5.7. I don't like cherry-picks, but no other way otherwise to have it back. What would you prefer?

Revision Contents
Changeset List

			Path	Packages
M			Modules/about-distro/src/Module.ui (3 lines)
M			Modules/fileindexermonitor/kcm_fileindexermonitor.desktop (3 lines)
M			Modules/fileindexermonitor/package/metadata.desktop (2 lines)
M			org.kde.kinfocenter.desktop (1 line)