]> git.tdb.fi Git - ext/subsurface.git/commit
Stop relying on gtk to track which dives are selected
authorDirk Hohndel <dirk@hohndel.org>
Thu, 16 Aug 2012 23:31:53 +0000 (16:31 -0700)
committerDirk Hohndel <dirk@hohndel.org>
Thu, 16 Aug 2012 23:47:49 +0000 (16:47 -0700)
commit7cf0f6d5e19f57449078901a3730f9e40de3a012
treec48f1e9b821a0e1274b99181b4eb8bfe6c39bca9
parente6ecddfa3d17901847474b67b6121c0c8f56f078
Stop relying on gtk to track which dives are selected

We spend way too much effort trying to get gtk to manage the dives that
are selected. The straw that broke the camel's back is that gtk forces us
to expand any nodes that we want to select - so selecting a summary entry
for a dive trip forced us to expand all the dives in the dive trip. Which
as Linus pointed out really sucked from a user experience.

So instead we now completeley ignore gtk's weird idea of what is selected
and what isn't and simply track things ourselves. We still need to play
some games with gtk to make sure that the correct rows are SHOWN as
selected, but still, the overall code seems much cleaner.

This commit contains a bunch of debugging code that is ifdef'ed out -
this is extremely useful to make sure I didn't mess anything up, but
eventually I'll want to remove that again as it just looks ugly in the
code.

Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
divelist.c
statistics.c