ToolbarApplicationHeader layout
ClosedPublic

Authored by mart on Mar 7 2018, 4:44 PM.

Details

Summary

layout, based on some kmail mockups as follows:

  • Title left aligned
  • main action buttons right-aligned
  • contextualactions in a menu
Test Plan

attached some screenshots on how it works in several different cases

Diff Detail

Repository
R169 Kirigami
Branch
phab/toolbartweak
Lint
No Linters Available
Unit
No Unit Test Coverage
mart created this revision.Mar 7 2018, 4:44 PM
Restricted Application added a project: Kirigami. · View Herald TranscriptMar 7 2018, 4:44 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
mart requested review of this revision.Mar 7 2018, 4:44 PM
mart changed the visibility from "Public (No Login Required)" to "No One".Mar 7 2018, 4:44 PM
mart retitled this revision from some layouting tweaks to ToolbarApplicationHeader layout.Mar 7 2018, 4:48 PM
mart edited the summary of this revision. (Show Details)
mart edited the test plan for this revision. (Show Details)
mart added a reviewer: Kirigami.
mart changed the visibility from "No One" to "Public (No Login Required)".Mar 7 2018, 4:55 PM
mart updated this revision to Diff 28960.Mar 7 2018, 5:01 PM
mart edited the summary of this revision. (Show Details)
  • paddings fixes
ngraham added a subscriber: ngraham.Mar 7 2018, 5:23 PM

+1, this is a huge visual improvement in my eyes.

apol accepted this revision.Mar 7 2018, 5:27 PM
apol added a subscriber: apol.
apol added inline comments.
src/styles/org.kde.desktop/Units.qml
101

Unrelated change.

This revision is now accepted and ready to land.Mar 7 2018, 5:27 PM
This revision was automatically updated to reflect the committed changes.