Sune Foldager
2011-11-23 14:35:41 UTC
Ok, this is a stab at making the stand-alone commit dialog support MQ the same way the embedded one does. It basically uses the same approach, but places the combo-button in the "dialog button bar position", as the Commit button has in that dialog, instead of at the top right.
I have tested all the scenarios I can come up with, so I am fairly sure it works, but someone should have a second look at the code. Some comments:
- The commit.commitButtonEnable.connect(commitbtn.setEnabled) line I moved, is not really used after the change, since the only situation the button was disabled, was when we had an MQ patch at the top. This will now trigger a proper MQ combo-button. I left it in anyway, but we might get rid of it.
- I made the 'Commit' text bold for the non-MQ case, to conform with the MQ button as well as the Commit button in the embedded version.
- The rev='.' is necessary to correctly trigger QRefresh instead of QNew as default action when there's a patch on top.
-Sune
I have tested all the scenarios I can come up with, so I am fairly sure it works, but someone should have a second look at the code. Some comments:
- The commit.commitButtonEnable.connect(commitbtn.setEnabled) line I moved, is not really used after the change, since the only situation the button was disabled, was when we had an MQ patch at the top. This will now trigger a proper MQ combo-button. I left it in anyway, but we might get rid of it.
- I made the 'Commit' text bold for the non-MQ case, to conform with the MQ button as well as the Commit button in the embedded version.
- The rev='.' is necessary to correctly trigger QRefresh instead of QNew as default action when there's a patch on top.
-Sune