HomePhabricator
Diffusion MITK 3ed547900326

removed mbilogo.h and replaced the default mechanic and default logo also…

Description

removed mbilogo.h and replaced the default mechanic and default logo also: - cleaned up some color definitions - got rid of unneeded icon files

Details

Provenance
kirchnthAuthored on Sep 1 2017, 5:03 PM
kirchnthPushed on Sep 1 2017, 5:02 PM
Parents
rMITK56e8743dfb48: Merge branch 'T22775-dcmtk-update'
Branches
Unknown
Tags
Unknown

Event Timeline

A couple of comments

/Modules/QtWidgets/src/QmitkStdMultiWidget.cpp
324

Why?
29/256 is going to be 0, in that case you wouldn't have had to change anything.

334

s.o.

/Plugins/org.mitk.gui.qt.stdmultiwidgeteditor/src/QmitkStdMultiWidgetEditor.cpp
339–341

This is rather cryptic. It seems it was that way before, but we should at least add the comment:

we need to disable the logo first, otherwise setting a new logo will have
no effect due to how mitkManufacturerLogo works...

from below here as well

29/256 was actually set as 0.11sth as i intended in my build ... but i agree that specifying float is safer