<div dir="ltr"><div><div>William, <br><br></div>The patch is awesome, thanks.<br></div><div>Rick - Agreed, but this can be easily added.<br><br></div><div>Tomaz<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Nov 20, 2016 at 10:33 PM, Rick Walsh <span dir="ltr"><<a href="mailto:rickmwalsh@gmail.com" target="_blank">rickmwalsh@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Hi Willem,<br><br></div>I think this looks great, and very useful for dive and/or surface photographers.<br><div><div><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On 19 November 2016 at 23:44, Willem Ferguson <span dir="ltr"><<a href="mailto:willemferguson@zoology.up.ac.za" target="_blank">willemferguson@zoology.up.ac.<wbr>za</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Subject: [PATCH] Provide photos summary on dive list<br>
<br>
1) Add an extra column to dive list, just left of Locality field.<br>
2) For each dive, give summary of photos as follows:<br>
i) no photos: no icon in that column<br>
ii) photos taken during dive: show icon of fish<br>
iii) photos taken before/after dive: show icon of sun<br>
iv) photos taken during as well as before/after dive: show<br>
icon with both fish and sun<br></blockquote></span><div>The icons are very clear as to what is during dive, and what is not. However, I think they could be clearer to show that they represent photographs, especially if the column heading is not going to have a full title. I could be worth surrounding the icons with a picture frame.<br><br></div><div>Cheers,<br><br></div><div>Rick<br></div></div></div></div></div></div>
<br>______________________________<wbr>_________________<br>
subsurface mailing list<br>
<a href="mailto:subsurface@subsurface-divelog.org">subsurface@subsurface-divelog.<wbr>org</a><br>
<a href="http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface" rel="noreferrer" target="_blank">http://lists.subsurface-<wbr>divelog.org/cgi-bin/mailman/<wbr>listinfo/subsurface</a><br>
<br></blockquote></div><br></div>