D12933: Update the MultipleStaff view to automatically add more staffs when more notes are added.

[Aman Kumar Gupta]

amankumargupta added inline comments.

INLINE COMMENTS

> jjazeix wrote in MultipleStaff.qml:81
> is there a way to not have to save/load every time we add a staff?

I think it's not. Cause when we increase the number of staves (model), the entire model gets refreshed and the notes are erased. So we need to store the notes temporarily and when then put it back into the model.

Regarding saving, we don't actually save it. We just temporarily store it in var melody and then feed this data to loadFromData() to insert them back into the staves.

> jjazeix wrote in MultipleStaff.qml:83
> there was a else before, it is normal that it is removed?

yes it is normal. Previously since we had fixed number of staff so if the staff were full, we would return else we increment the currentStaff. But now since we don't have limitations on staff numbers, we always increment it when the last staff is full (as we add a new staff at the end).

REPOSITORY
  R2 GCompris

REVISION DETAIL
  https://phabricator.kde.org/D12933

To: amankumargupta, #gcompris_improvements, jjazeix
Cc: jjazeix, kde-edu, #gcompris_improvements, himanshuvishwakarma, harrymecwan, ganeshredcobra, nityanandkumar, echarruau, rahulyadav, narvaez, scagarwal, apol, timotheegiet, hkaelberer, bcoudoin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20180518/8822d48e/attachment.html>