Review Request 119905: Parley: Fix bugs: 338302 and 338417 title bar [modified] tag behavior is inconsistent.

Andreas Xavier andxav at zoho.com
Fri Aug 22 19:55:04 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119905/
-----------------------------------------------------------

(Updated Aug. 22, 2014, 7:55 p.m.)


Review request for KDE Edu.


Changes
-------

Fixed comments and changed function name to highlight its use.


Bugs: 338302 and 338417
    http://bugs.kde.org/show_bug.cgi?id=338302
    http://bugs.kde.org/show_bug.cgi?id=338417


Repository: parley


Description
-------

Symptoms:
The [modified] in the title bar was inconsistent.

Problems:
1. The title bar was only connected to the dirty bit in the doc from the Editor.  Going directly welcome -> stats did not connect the title bar to the dirty bit.
2. Whenever a new document was created, new collection, welcome screen etc. the title bar was disconnected.
3. In several places setModified was called on the kvocdoc when either nothing changed or the kvocdoc should know better.

Solution:
The title bar is ParleyMainWindow's title bar.  ParleyMainWindow also know when the document is updated.  I created a  updateParleyDocument so that ParleyMainWindow always knows when the doc changes and slotFileNew so that anyone can ask ParleyMainWindow to update its own title bar.

Not fixed: Parley still directly calls the setModified in kvocdoc when it starts practice.  kvocdoc needs to be fixed before that is removed.


Diffs (updated)
-----

  src/editor/editor.cpp 4fac862 
  src/parleydocument.cpp 0ebeca1 
  src/parleymainwindow.h 98a94ad 
  src/parleymainwindow.cpp 9723ee1 
  src/practice/practicemainwindow.cpp 3008501 
  src/settings/documentproperties.cpp 02cbf7b 
  src/settings/languageproperties.cpp e9b171f 
  src/vocabulary/vocabularymodel.cpp abf0db9 
  src/welcomescreen/welcomescreen.cpp d24f444 

Diff: https://git.reviewboard.kde.org/r/119905/diff/


Testing
-------

Clicked from welcomescreen -> stats -> practice -> exit stats
Clicked from welcomscreen -> stats -> editor (edited) -> practice
Clicked from welcomscreen -> stats -> editor (no edit) -> practice
Created a new collection from welcome screen
Changed doc properties from stats->File->Properties.

All of these cases worked correctly


Thanks,

Andreas Xavier

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20140822/009f3bc3/attachment.html>


More information about the kde-edu mailing list