#6865: [with patch, partial positive review] Use templates for CSS
-------------------------+--------------------------------------------------
 Reporter:  timdumol     |       Owner:  boothby                 
     Type:  enhancement  |      Status:  new                     
 Priority:  minor        |   Milestone:  sage-4.1.2              
Component:  notebook     |    Keywords:  notebook css stylesheets
 Reviewer:               |      Author:  Tim Dumol               
   Merged:               |  
-------------------------+--------------------------------------------------

Comment(by mpatel):

 Changes in patch v3:

  * `css.py`: `if os.path.exists(user_css):` --> `if
 os.path.exists(user_css_path):`
  * `css.py`: `+` --> `os.path.join()`
  * `css.py`: html -> HTML in docstring
  * `main.css`: {% ->  {%-

 Sorry about the trivial stuff.  I just noticed the
 [http://jinja.pocoo.org/2/documentation/templates#whitespace-control
 whitespace control] part of the Jinja2 docs.

 This is a positive review from me, but someone should review my changes.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6865#comment:10>
Sage <http://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 post to this group, send email to [email protected]
To unsubscribe from this group, send email to 
[email protected]
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to