[I7] Table of Contents

[Andrew Hunter]

I've uploaded another version of the application to the same place as  
before. This should implement Graham's suggested changes, and fix the  
editing problem, which should mean things are now reliable.

I've also decided to start uploading source code to <http://www.logicalshift.co.uk/etc/Inform-src.zip 
 > to go with the -dev archives, so it should always be possible to  
get the latest source code.

On 15 Jan 2008, at 00:37, Graham Nelson wrote:
>
> There are two noteworthy things here:
>
> (a) the drop-down selection line from Part II ends at the foot
> of the TOC with no black dot of a subsequent section to
> anchor it to, and
>
> (b) there's no line down from Chapter Z to Chapter E.
> Chapter Z has no sections, so there aren't multiple
> headings to be selected.
>
> I think both of these cases are handled correctly.

Something else to note here is that the line is shorter when it is not  
connecting to a black dot (if it were the same length, it would  
overlap the following text.

> When there are no headings, the TOC now displays the placeholder
> text nicely. But
>
> (i) it appears at a random height in the panel, which is disconcerting
> (sometimes in the middle, but not always, and I don't know why)
> - it would be better to have it always up at the top of the panel,
> heading the page so to speak. In particular, it looks very odd
> now if there's a title but no headings in the source text, and
> that's actually the most likely way a new user will see it, since
> newly created projects have title but no headings.

Annoyingly, Apple didn't add the API to calculate the height of a  
section of text given it's width until 10.4, so I was cheating to make  
the text look vaguely centered - drawing it 40 pixels above the center  
of the panel to ensure 10.3 compatibility.

I've changed this now to drawing text at the top.

> (ii) the point size used for this placeholder text is a little too
> small, I think. (Yes, it matches that used for headings, where in fact
> I think the point size is right -- but there's no reason this  
> placeholder
> text has to match that.)

OK, that's changed now.

> (iii) my remarks about height also apply to the text
>
> "No headings are visible at this level. Drag the slider below to the
> right to make the headings in your source code visible."
>
> which appears on some positions of the slider. (A good idea, this,
> but could we say "the source text" rather than "your source code"?)

I've changed this as well. If you want to play with the strings that  
are displayed here, you can change the Localizable.strings file in the  
application bundle. Make sure you send me anything you want made  
permanent.

Andrew.