#10637: Implement sage -sws2rst
------------------------------------------------------------+---------------
Reporter: nthiery | Owner:
jason, mpatel, was
Type: enhancement | Status:
needs_work
Priority: major | Milestone:
sage-5.7
Component: notebook | Resolution:
Keywords: ReST, worksheet | Work issues:
Report Upstream: Workaround found; Bug reported upstream. | Reviewers:
Nicolas Thiéry, Jason Grout, Karl-Dieter Crisman, Jason Bandlow, John Palmieri,
Simon King
Authors: Pablo Angulo, Karl-Dieter Crisman | Merged in:
Dependencies: #11080, #11459 | Stopgaps:
------------------------------------------------------------+---------------
Changes (by kcrisman):
* status: needs_review => needs_work
* reviewer: Nicolas Thiéry, Jason Grout, Karl-Dieter Crisman, Jason
Bandlow, John Palmieri => Nicolas Thiéry,
Jason Grout, Karl-Dieter Crisman, Jason
Bandlow, John Palmieri, Simon King
* work_issues: answer questions, math formatting, lists, maybe absolute
paths? =>
Comment:
> It is bold and cursive because I clicked the bold and cursive buttons in
the editor that comes with the notebook. Since bold and cursive is
supported by rst, my uneducated guess is that a converter should be able
to deal with it.
Yes, that does make sense. I guess the question is whether "premature
optimization is the root of all evil" or not. In this case, I think that
having an sws2rst converter in Sage that at least converts ''some'' stuff,
but can't handle styles, is worse than having no sws2rst converter at all.
I have found this to be extremely useful in its current state, so I'm not
sure it's worth postponing it indefinitely for this reason (as opposed to
checking the code itself, which I have lamentably not had time to finish
off, but which I should do today).
I think that the problem is that
{{{
<p style="padding-left: 30px;">
}}}
is how you achieved your indentation. That's an HTML style, not
whitespace.
Okay, on further investigation, what you must have done is clicked the
"indent" button. Usually I only use this with lists, which creates more
levels of lists (and which sws2rst already handles correctly).
Pablo, do you think that finding this particular thing? That is, if we
have
{{{
style="padding-left: 60px;"
}}}
where 60 could be any positive integer multiple of 30, as a specific thing
in the tag `<p ...>`, could that specific thing be searched for and then
replaced with an equivalent amount of whitespace (maybe four ` ` for
instance, per 30px)? Certainly it would be nightmarish to try to make
that into an appropriate amount of lists, and in any case we don't want
the list bullets in that case.
----
A question for Simon - did the Sage root and scripts patches apply fine on
a recent beta of 5.7? I assume so, just checking with respect to
rebasing.
----
Okay, for positive review we need the following, in my opinion. The
previous work issues were dealt with long ago.
* Still to check that regex, though apparently it's working fine
* Add levels `<h5>` and `<h6>` (see
[http://www.w3.org/TR/html401/struct/global.html#h-7.5.5 here], h6 is also
actually supported by TinyMCE in Sage)
* Decide what to do with address tag (possibly turn it into pre tag? just
an idea) and do that
* Decide whether it's feasible to replace the specific style of
indentation mentioned above, and do it if so
* Make an updated pull request for sagenb (luckily not a problem with
respect to rebasing, since we only add new files)
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10637#comment:123>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.
