Update AtCore::PrintWidget
ClosedPublic

Authored by rizzitello on Apr 14 2018, 7:42 PM.

Diff Detail

Repository
R232 AtCore
Branch
tempPrintUpdate
Lint
No Linters Available
Unit
No Unit Test Coverage
rizzitello requested review of this revision.Apr 14 2018, 7:42 PM
rizzitello created this revision.
rizzitello edited the summary of this revision. (Show Details)Apr 14 2018, 7:43 PM
rizzitello added a project: Atelier: AtCore.
rizzitello added a subscriber: Atelier: AtCore.
patrickelectric requested changes to this revision.Apr 15 2018, 6:20 PM
patrickelectric added inline comments.
src/widgets/printwidget.cpp
102

80 ? This should be the alignment right ?

src/widgets/printwidget.h
99

Alphabetic order

This revision now requires changes to proceed.Apr 15 2018, 6:20 PM
rizzitello updated this revision to Diff 32206.Apr 15 2018, 6:34 PM
rizzitello marked 2 inline comments as done.
rizzitello edited the summary of this revision. (Show Details)
  • update for widget changes
rizzitello added inline comments.Apr 15 2018, 6:35 PM
src/widgets/printwidget.cpp
102

Naw we want it to use 80% of the layout space

patrickelectric accepted this revision.Apr 15 2018, 7:31 PM
This revision is now accepted and ready to land.Apr 15 2018, 7:31 PM
rizzitello closed this revision.Apr 15 2018, 7:55 PM