Update of the Photo Section

[Stephanie Mardell]

Hi Phil,

Thanks for the feedback - I will update the patch.

In regards to providing a "patch". Do you prefer small section patches
or provide one large patch that updates the whole of the Photos section?

Stephanie

On Sun, 2008-09-14 at 00:20 +0100, Phil Bull wrote:
> Hi Stephanie,
> 
> On Thu, 2008-09-11 at 17:34 -0700, Stephanie Mardell wrote:
> > Here are my first set of changes to the Photos Section. At this stage
> > I have only updated the Import Photos section (Section 5.1) - I do not
> > want to get too far ahead without someone reviewing what I have done
> > to check it for correctness.
> 
> The patch looks good, thanks for your work on it so far. In particular,
> I definitely prefer how the user is kept inside F-Spot rather than being
> told to manually copy their photos across if their camera isn't listed
> as an import source. The change of section title is a great idea too,
> I'd never even considered people importing from memory cards.
> 
> Here's some feedback on the rest of the patch:
> 
>       * "...F-Spot</ulink> is a full-featured..." - I think that
>         instructions on starting F-Spot should be provided in the
>         initial sentence.
>       * "~/Photos folder" - not all users will understand that the tilde
>         means "Home folder". I suspect that most will press Places ->
>         Home Folder -> Photos to get there, so maybe something more
>         representative of that route to the destination, e.g. "the
>         Photos folder, which can be found in your Home folder".
>       * "only create references to images stored in other folders" - I
>         don't think that this wording hints at the reason why you'd want
>         to use that option explicitly enough. Maybe something like "add
>         photos to the collection without copying them from their
>         existing location".
>       * "See the ... F-Spot User Guide" - F-Spot has built-in
>         documentation now, so we should try to link to that instead.
> 
> Thanks,
> 
> Phil
>