Rework of commitmodel and dialog

Review Request #126427 - Created Dec. 19, 2015 and submitted

Christian Ehrlicher
cgiboudeaux, dfaure

The Commit dialog had a problem with the 'hide new files' functionality. Internally two lists (visible & hidden files) were used. The handling was somehow broken so that some files disappeared after toggling. Short test: create three new files in a directory, push the hide new files button twice and you'll see only two of them again...). Within this work to fix this, I also changed other things:
- instead doing checked/unchecked handling inside CommitModel, use a separate CommitFilterModel to avoid the (faulty) handling of two lists inside CommitModel
- use QVector instead QList
- Enable/Disable the 'Revert/Diff Item' buttons depending on the current selected item
- Toggle text in Hide new Items button

Did some test commits with debug-lines and all seems to be ok. I also changed to sort order in the view to check if all mapTo/mapFromSource are correct.


  • 0
  • 6
  • 0
  • 6
Description From Last Updated
David Faure
Christian Ehrlicher
Review request changed

Status: Closed (submitted)

Change Summary:

Submitted with commit 5dcb34520f619fcf8c799826ff77d401eab41935 by Christian Ehrlicher to branch master.