-
-
Notifications
You must be signed in to change notification settings - Fork 535
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
Set up jsMath extensions, macros, etc., for the documentation #6673
Comments
Depends on #6614. |
comment:1
Attachment: trac_6673-jsmath_macros_docs.patch.gz Note: Depending on the outcome of this ticket, we may need to update the "scripts" patch at #6187. |
Reviewer: John Palmieri |
comment:2
With the patch, commands like Is there any way to generate the list of macros automatically, the way it is done in
I don't see anything helpful in the Sphinx documentation, unfortunately. So instead, I'm attaching a small patch to the file For the file jsmath_sage.js, it looks like a modified version of easy/load.js, but it's not the same as the one distributed with Sage (the one in local/notebook/javascript/jsmath/easy); there are white space differences, and there are differences below the line saying "DO NOT MAKE CHANGES BELOW THIS". Do I need to worry about this? Let me know, and if it's not a problem, I can give this a positive review. |
Attachment: trac_6673-referee.patch.gz apply on top of the other patch |
comment:3
Rather inconveniently, I described the patch, briefly, in a comment at #4714. I apologize for not adding some information here. With Sphinx 0.6.2 (cf. #6586), we might use static templates to insert the macros on-the-fly. A test:
I find Since we're not there yet, I give the "referee" patch a positive review. |
comment:4
I just noticed that jsMath complains about
|
comment:5
So the plan is to have this depend on #6586? (That's fine with me, I just want to make sure I understand.) |
comment:6
The forthcoming alternative patch inserts macros on-the-fly. It
[1] It seems that we can override the templates' default |
Attachment: trac_6673-jsmath_macros_docs_v2.patch.gz Template version. Depends on #6187, #6586. Apply only this patch. |
comment:7
I haven't added/fixed
|
comment:8
Re congroup_gammaH.html, if it's the "\trianglelefteq", then I think it can safely be replaced with "\leq". (It's a subgroup of an abelian group, so it's automatically normal.) |
comment:9
(Although \trianglelefteq should be available through the amssymbols package. Hmm.) |
comment:10
Should we add new macros in a separate ticket? If so, I can try to set up #4714 to use the same |
comment:11
Also: How should we set up tex2math in |
Attachment: trac_6673-jsmath_macros_docs_v3.patch.gz Updated template version. Added 'sage' theme. Apply only this patch. |
comment:13
Version 3:
Remarks:
|
This comment has been minimized.
This comment has been minimized.
comment:15
The code looks good. I've played around with it for a while, and it seems to do what it's supposed to. Now schilly can get to work modifying "theme.conf" to produce a custom Sage theme... |
comment:16
To the release manager: when merging this, please close #7204 also, since this resolves the problem reported there. |
Merged: sage-4.2.1.alpha0 |
Currently, the documentation uses a stock jsMath loader script. This excludes Sage-specific customizations, e.g., the notebook's default jsMath macros. The patch
inserts the settings via a template when the docs are built. The patch also sets up a 'sage' HTML theme (cf. Sphinx docs) for later customization.
See #4714 for a "notebook" version.
CC: @haraldschilly
Component: documentation
Author: Mitesh Patel
Reviewer: John Palmieri
Merged: sage-4.2.1.alpha0
Issue created by migration from https://trac.sagemath.org/ticket/6673
The text was updated successfully, but these errors were encountered: