ZTree.com  | ZEN  | About...  

 Index   Back

[Nit] Bookmark-Fix Wording   [Release]

By: Kim Henkel     Riverview, FL  
Date: Mar 19,2009 at 01:38
In Response to: [Nit] Bookmark-Fix Wording (John Gruener)

> > > Could you ever remove a current directory if it was not bookmarked? :)
> > No, but you could remove the current bookmark if the current directory was not bookmarked ;-)
> Well, I never thought of it as the "current" bookmark. Only the "most
> recent" bookmark.

Fair enough - I was just having some fun with the language ;-)

> > > I think better wording would be:
> > > Fixed - Ctrl-[ - Remove bookmark only if that directory is current
> > I'm just not sure it's clear what 'that directory' refers to?
> Hmm... It's the directory that that bookmark refers to. :)

But which bookmark? ;-)

As an example, say I have bookmarks at A, B & C, and I use [ to navigate from C to B to A. If I then navigate (not using bookmarks) to D, what should Ctrl-[ do?
The answer is nothing. The problem is that previously A was removed.

In this scenario, which is what prompted Olivier to report it:
"Remove bookmark only if that directory is current"
doesn't say anything about which bookmark might be effected.

> > so perhaps this would be enough:
> > Change - Ctrl-[ (DW) - Remove bookmark for current directory
> OK. Sorry for much ado about a minor point, but didn't it always remove
> the bookmark for the current directory?

No, the current directory played no part in it - it always removed the most recently visited bookmark!

> What it used to do, and no longer does, was the remove the "most
> recent" bookmark if the current directory was not bookmarked.

Current directory was irrelevant - what it did was 'remove the "most recent" bookmark' - period.

> So, since it's documented to only "remove the current directory from
> the list", then I would consider this a "fix" rather than a "change".

But the documentation was wrong ;-)
It should have said:
"To remove the most recently visited directory from the list, press Ctrl-["

The documentation now needs to say something like:
"To remove the current directory from the list, if it exists, press Ctrl-["

> How about replacing my words "that directory" with "the directory to
> which it refers" like this:
> Fixed - Ctrl-[ (DW) - Remove bookmark only if the directory to which it
> refers is current
>
> Or a totally different approach:
> Fixed - Ctrl-[ (DW) - Don't remove most recent bookmark if it's not the
> current directory
>
> Each a bit long, but not the longest lines in there. ;-)

Sorry, but I think these both still miss the point - the context of the function now is the 'current directory' - not any particular bookmark.
We are not 'choosing' a bookmark (by whatever means) then deciding whether or not to remove it - we are taking the current directory, and looking to see if it is bookmarked, and only then if it is, do we remove it.


Kim

1,026 views      
Thread locked
 

Messages in this Thread

 
96,637 Postings in 12,231 Threads, 350 registered users, 71 users online (0 registered, 71 guests)
Index | Admin contact |   Forum Time: Mar 29, 2024 - 10:04 pm UTC  |  Hits:62,424,963  (39,749 Today )
RSS Feed