diff options
| author |  Dirk Hohndel <dirk@hohndel.org> | 2011-12-07 11:58:16 -0800 |
|---|---|---|
| committer | Dirk Hohndel <dirk@hohndel.org> | 2011-12-07 15:11:06 -0800 |
| commit | af290d5eb281662d336b8186a9df1a489d78ec9d (patch) | |
| tree | 66305a529612c97d6f262c24ce6d024022546f2b /parse-xml.c | |
| parent | ce6c3ee56d372e2ef8386a065fbf4bce4cfa0ffc (diff) | |
| download | subsurface-af290d5eb281662d336b8186a9df1a489d78ec9d.tar.gz | |
Add typical 0 to 5 star rating for dives
This works ok-ish, but doesn't allow us to click on the stars and edit
them in the divelist, which a user might expect to be able to do - in
most "star rating UIs" you simply click on the n-th star to set that
rating. Here you need to edit the dive and pick the rating from a drop
down menu.
Minor oddity: you can actually (if you force it) write anything you want
into the star rating. But anything that isn't one of the predefined
strings simply results in a zero star rating.
Overall the UI feels a bit... forced. But I think this is quite useful
anyway.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'parse-xml.c')
| -rw-r--r-- | parse-xml.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/parse-xml.c b/parse-xml.c index b8f679324..82b10a47e 100644 --- a/parse-xml.c +++ b/parse-xml.c @@ -1024,7 +1024,8 @@ static void try_to_fill_dive(struct dive **divep, const char *name, char *buf) return; if (MATCH(".buddy", utf8_string, &dive->buddy)) return; - + if (MATCH(".rating", get_index, &dive->rating)) + return; if (MATCH(".cylinder.size", cylindersize, &dive->cylinder[cylinder_index].type.size)) return; if (MATCH(".cylinder.workpressure", pressure, &dive->cylinder[cylinder_index].type.workingpressure)) |