search

ivoflipse / Pawlabeling

Tool for processing and analyzing pressure measurements
Other
18 stars 1 forks source link

Remove contact_label == -1 for the current contact #99

Closed ivoflipse closed 10 years ago

ivoflipse commented 10 years ago

Because I already keep track of the current contact with its index, there's no need to change its label.

I'm also considering adding some kind of index, which I can use to lookup items in the measurement tree, without having to traverse it. If its stored in the model, it can be shared between widgets.

Because if I don't change the label before switching contacts or worse, measurements, it will leave other contacts marked as Current, even if they're not.

I also think removing Current would be a good idea, because of the issues it might create. Simply have some visual feedback instead.

ivoflipse commented 10 years ago

Perhaps this is a reason to start using a QAbstractItemModel after all, because it should automatically handle updating of information and what not. Possibly the model should even be able to return me the current item.

ivoflipse commented 10 years ago

This is solved, I'm moving the QAbstractItemModel thing into its own issue