Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

broken links from website index to tutorial, constructions, etc. #6485

Closed
qed777 mannequin opened this issue Jul 8, 2009 · 6 comments
Closed

broken links from website index to tutorial, constructions, etc. #6485

qed777 mannequin opened this issue Jul 8, 2009 · 6 comments

Comments

@qed777
Copy link
Mannequin

qed777 mannequin commented Jul 8, 2009

Build the HTML documentation, including the website, and navigate to $SAGE_ROOT/devel/sage/doc/output/html/en/index.html. Clicking on a link to an individual document yields a directory listing, instead of the expected index page.

Component: documentation

Author: Mitesh Patel

Reviewer: David Loeffler

Merged: sage-4.1.1.alpha0

Issue created by migration from https://trac.sagemath.org/ticket/6485

@qed777 qed777 mannequin added c: documentation labels Jul 8, 2009
@loefflerd
Copy link
Mannequin

loefflerd mannequin commented Jul 13, 2009

comment:1

Attachment: trac_6485_website_links.patch.gz

This ticket looks like a duplicate of #5550 to me. Since mpatel's already uploaded a patch here, I suggest we close #5550 as a duplicate and keep this one open.

@loefflerd
Copy link
Mannequin

loefflerd mannequin commented Jul 13, 2009

Reviewer: David Loeffler

@loefflerd
Copy link
Mannequin

loefflerd mannequin commented Jul 13, 2009

comment:2

Works for me.

@loefflerd loefflerd mannequin added the s: positive review label Jul 13, 2009
@loefflerd loefflerd mannequin added this to the sage-4.1.1 milestone Jul 14, 2009
@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Jul 18, 2009

Merged: sage-4.1.1.alpha0

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Jul 18, 2009

comment:4

The patch fixes the index.html linking problem, but only for the page

SAGE_ROOT/devel/sage/doc/output/html/en/index.html

There's also a "website" which can be built using

sage -docbuild website html

and the same problem is still with the page

SAGE_ROOT/devel/sage/doc/output/html/en/website/index.html

But I doubt we need "website" at all. Its purpose is to link to the other 8 standard documents. This is already achieved with

SAGE_ROOT/devel/sage/doc/output/html/en/index.html

So I'm closing this ticket as fixing the linking problem in the page

SAGE_ROOT/devel/sage/doc/output/html/en/index.html

Feel free to open another ticket to fix or delete the "website" page.

@sagetrac-mvngu sagetrac-mvngu mannequin closed this as completed Jul 18, 2009
@qed777
Copy link
Mannequin Author

qed777 mannequin commented Jul 19, 2009

comment:5

I think sage -docbuild website html builds the top-level index.html in website/ and just copies the output up one directory level. (We could delete the original afterward.) Perhaps I should have put "web site" in the summary instead of "website." I apologize, if I'm mistaken.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

0 participants