Opened 4 years ago

Closed 4 years ago

#17934 closed enhancement (fixed)

Ref: reorganize main TOC

Reported by: mmezzarobba Owned by:
Priority: major Milestone: sage-6.6
Component: documentation Keywords: reference
Cc: Merged in:
Authors: Marc Mezzarobba Reviewers: Julien Puydt
Report Upstream: N/A Work issues:
Branch: f2de8d8 (Commits) Commit: f2de8d8fb37bfcaac0050365e817fc5eb7ef9240
Dependencies: Stopgaps:

Description

Reorganize the main table of contents of the reference manual

Change History (7)

comment:1 Changed 4 years ago by mmezzarobba

  • Status changed from new to needs_review

comment:2 Changed 4 years ago by git

  • Commit changed from 599451ba70a3bb39c1349870c22459a8360010d7 to f2de8d8fb37bfcaac0050365e817fc5eb7ef9240

Branch pushed to git repo; I updated commit sha1. New commits:

f2de8d8Remove todo list from reference manual.

comment:3 Changed 4 years ago by ncohen

HMmmm... It is pretty hard to understand what you do from just looking at the commit diff :-/

comment:4 follow-up: Changed 4 years ago by ncohen

What exactly did you want to reorganize?

Nathann

comment:5 in reply to: ↑ 4 Changed 4 years ago by mmezzarobba

Replying to ncohen:

What exactly did you want to reorganize?

I reordered the links to the individual chapters, renamed some of them, and organized them into subsections. Unless I made a mistake, there is no new link, and the only deleted link is the one to the todo list. This should be easy to check with git diff | grep '+\*' | sort or something similar. Knowing that, I think just comparing the full tables of contents (old and new) should give you a pretty clear picture of the changes.

comment:6 Changed 4 years ago by Snark

  • Reviewers set to Julien Puydt
  • Status changed from needs_review to positive_review

Looks good.

comment:7 Changed 4 years ago by vbraun

  • Branch changed from u/mmezzarobba/ref-toc to f2de8d8fb37bfcaac0050365e817fc5eb7ef9240
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.