feature requests for loggerhead

[Robey Pointer]

On 25 Jun 2007, at 5:48, Kent Gibson wrote:

> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
>
>>
>> I like the condensed changelog listing, very nice.  I tweaked this
>> a bit to add the committer's name, etc.  My variant branch is here:
>>
>>
>> http://www.lag.net/robey/code/loggerhead.robey.kg/
>>
>> I didn't want to merge it right away in case my tweaks messed up
>> the UI, but if it still looks basically okay to you two, I'll merge
>> it in.
>>
>
> That looks ok to me, though I still prefer having the author/commiter
> in the expanded section and not the collapsed form.  Not a high enough
> S/N.

For some projects (like bzr.dev) it's useless info, because it'll  
always be PQM.  But for other projects, it's one of the most  
interesting pieces of info IMHO.  Maybe it should be a project option  
to flip it on/off.  Or maybe if every revision on the page has the  
same committer name, loggerhead could drop that info out of the top  
line and put it into the expanded section.  Actually that'd be pretty  
cool.  I'm gonna try that...

(later)  Hmm, that seems to work well.  Any objections?


>
>> Kent, I think you're right to start removing those little arrows
>> (the "-> files" things).  I was looking for a way to indicate "hey,
>>  these are links!" but it's pretty obvious, I guess.  Maybe the
>> list of links on the "revision" page should use bullets or
>> something. The arrows just look odd.
>>
> I was going to have more of a go at the revision and inventory pages
> as well but I got side tracked fixing bzr log, then real work, and
> never got back to it.
>
> I find the little link arrows to be superfluous.  If you are not sure
> what the word is there for the tooltip will (usually) tell you.
>
> I would just drop the arrows from the list on the top of the revision
> page.  It's pretty clear it is a list of actions related to the page.
> Maybe stick them in a toolb[ar|ox].

Yeah, I think the list of potential actions is sufficient.  The  
arrows don't help.  The wording could be improved too.  I showed the  
list of actions to a friend and he couldn't really guess what they  
actually did.


> I'd also remove the ones in the "files modified", and fix the diff
> links so they also work in unified view - at the moment they only work
> in side-by-side.  The anchor that the link refers to is currently
> within the sbs block and so isn't accessible when that block is hidden
> in unified view.

Ahh, it was a trickier problem.  The anchor existed in both blocks,  
but that doesn't fool Camino/Mozilla, and it only uses the first  
one.  I've worked around that by making two copies of the modified- 
files list, each with different anchor links, and giving the anchors  
different names.

robey