From 02bd99697b9907da3ddf860bb25618a074fbc98b Mon Sep 17 00:00:00 2001
From: Victor Shnayder <victor@mitx.mit.edu>
Date: Mon, 29 Oct 2012 07:49:33 -0400
Subject: [PATCH] Clean up section ids and classes

- all inputtypes should have id inputtype_${id}
- should have class capa_inputtype
---
 common/lib/capa/capa/templates/crystallography.html | 2 +-
 common/lib/capa/capa/templates/textinput.html       | 8 +-------
 common/lib/capa/capa/templates/vsepr_input.html     | 2 +-
 3 files changed, 3 insertions(+), 9 deletions(-)

diff --git a/common/lib/capa/capa/templates/crystallography.html b/common/lib/capa/capa/templates/crystallography.html
index 71578f1fa0..ee9b249d3d 100644
--- a/common/lib/capa/capa/templates/crystallography.html
+++ b/common/lib/capa/capa/templates/crystallography.html
@@ -1,4 +1,4 @@
-<section id="textinput_${id}" class="textinput" >
+<section id="inputtype_${id}" class="capa_inputtype" >
      <div id="holder" style="width:${width};height:${height}"></div>
     <div class="script_placeholder" data-src="/static/js/raphael.js"></div>
     <div class="script_placeholder" data-src="/static/js/sylvester.js"></div>
diff --git a/common/lib/capa/capa/templates/textinput.html b/common/lib/capa/capa/templates/textinput.html
index 14d54d4cde..d2059fdc58 100644
--- a/common/lib/capa/capa/templates/textinput.html
+++ b/common/lib/capa/capa/templates/textinput.html
@@ -1,12 +1,6 @@
 <% doinline = "inline" if inline else "" %>
 
-<%
-# TODO: 
-# Is id inputtype_${id} vs textinput_${id} important?
-# Is class capa_inputtype vs textinput important?
-# should really just use one.
-%>
-<section id="${'inputtype' if do_math else 'textinput'}_${id}" class="${'text-input-dynamath capa_inputtype' if do_math else 'textinput'} ${doinline}" >
+<section id="inputtype_${id}" class="${'text-input-dynamath' if do_math else ''} capa_inputtype ${doinline}" >
 
     % if preprocessor is not None:
     <div class="text-input-dynamath_data" data-preprocessor="${preprocessor['class_name']}"/>
diff --git a/common/lib/capa/capa/templates/vsepr_input.html b/common/lib/capa/capa/templates/vsepr_input.html
index 5194551d50..a6bd465554 100644
--- a/common/lib/capa/capa/templates/vsepr_input.html
+++ b/common/lib/capa/capa/templates/vsepr_input.html
@@ -1,4 +1,4 @@
-<section id="vsepr_input_${id}" class="textinput" >
+<section id="inputtype_${id}" class="capa_inputtype" >
      <table><tr><td height='600'>
     <div id="vsepr_div_${id}" style="position:relative;" data-molecules="${molecules}" data-geometries="${geometries}">
     <canvas id="vsepr_canvas_${id}" width="${width}" height="${height}">