Remove label of PrintProgressWidget
AbandonedPublic

Authored by laysrodrigues on Jun 3 2017, 7:24 PM.

Details

Summary

Signed-off-by: Lays Rodrigues <laysrodriguessilva@gmail.com>

Update xml of gui setup

  • Add toolbox option to show/hide main toolbox
  • Remove push option since I moved this widget to the toolbox

Signed-off-by: Lays Rodrigues <laysrodriguessilva@gmail.com>

Refactor of MainWindow

  • Remove initLocalVariables method
  • Update constructor to setup ConnecSettingsDialog
  • Add new connections on initConnectionsToAtCore, also

adding comments

  • Add new setup on initWidgets for the docks, also adding comments
  • On SetupActions:
    • update _connect action to also close the connection
    • Add a connect to update the toggle status of the _connect action when a connection isn't sucessful.
    • Add comments
    • Add actions for the plotDock, toolboxDock
    • Remove action for the pushWidget
  • HandlePrinterStatusChanged:
    • Change groupBox to toolBoxTaWidget
    • Change printProgressWidget to printProgressDockWidget
  • Refactor of all layout on the mainwindow, adding the docks and a frame

to simulate the 3DView while ins't done.

Signed-off-by: Lays Rodrigues <laysrodriguessilva@gmail.com>

Diff Detail

Branch
part3
Lint
No Linters Available
Unit
No Unit Test Coverage
laysrodrigues created this revision.Jun 3 2017, 7:24 PM
patrickelectric added inline comments.Jun 4 2017, 3:16 AM
src/mainwindow.cpp
50

Why in some places it have:

/*
  *  xxx
  */

and in others:
/* xxx */
Be consistent, just use /* xxx */

rizzitello requested changes to this revision.Jun 4 2017, 4:12 AM
rizzitello added inline comments.
src/mainwindow.cpp
49

please use single line comment for slingle comment line

//Start Connection to the 3DPrinter
112

This is not a KXMLGui issue as i had them working correctly before with KXMLGui in the Test client.
Please fix this.

122

These should not be needed. I think they are both on by default

217

use the QDockWidget's ToggleView Action Action instead

action = actionCollection()->addAction(QString("ActionNameFromRcFile),ui->plotDockWidget->toggleViewAction);

this is the proper actions to show / hide docks with KXMLGUI

This revision now requires changes to proceed.Jun 4 2017, 4:12 AM
laysrodrigues abandoned this revision.Jun 4 2017, 5:35 PM