translator: parallel text view layout adjustments

And some other small code changes.

2 files changed, 12 insertions, 4 deletions

diff --git a/src/www/translator/translator.css b/src/www/translator/translator.css
index de23ab47f..39763e3b6 100644
--- a/src/www/translator/translator.css
+++ b/src/www/translator/translator.css
@@ -12,7 +12,7 @@ div.pagehead {
 }
 table.menubar td { padding: 5px; }
 table.menubar dl, td.options > div > dl, dl.popupmenu {
-    z-index: 1;
+    z-index: 2;
     display: none; position: absolute;
     background: white; color: black;
     border: 1px solid black;
@@ -44,7 +44,7 @@ table.segments { margin-left: auto; margin-right: auto; }
 
 td.actions { padding: 1ex 1em 1ex 0.5em; }
 td.options { padding: 1ex 1em; }
-td.source, td.target {
+.segment td.source, .segment td.target {
     padding: 1ex;
     border-bottom: 2px solid #ccc;
     vertical-align: baseline;
@@ -65,9 +65,15 @@ td.source input[name=it], td.target input[name=it], textarea, input[name=punctch
 td.source input[name=it], td.target input[name=it], textarea { width: 100%; }
 
 table.paralleltexts td {
+    padding: 1ex;
     vertical-align: baseline;
     line-height: 130%;
 }
+table.paralleltexts td.source {
+    padding-right: 0.7em;
+    border-right: 2px solid #ccc;
+}
+table.paralleltexts td.target { padding-left:  0.7em; }
 
 label { font-family: sans-serif; }
 
diff --git a/src/www/translator/translator.js b/src/www/translator/translator.js
index 842404ddf..4f535a605 100644
--- a/src/www/translator/translator.js
+++ b/src/www/translator/translator.js
@@ -16,6 +16,7 @@ function Translator() {
     update_language_menu(this,"source")
     update_language_menu(this,"target")
     if(apertium) this.add_apertium()
+    //initialize_sorting(["TR"],["segment"])
     this.redraw();
 }
 
@@ -604,7 +605,8 @@ Translator.prototype.draw_document=function() {
 	var sources=mapix(fmt,map(src,doc.segments))
 	var targets=mapix(fmt,map(trg,doc.segments))
 	var drawing=[hdr,wrap_class("table","paralleltexts",
-				    tr([td(sources),td(targets)]))]
+				    tr([wrap_class("td","source",sources),
+					wrap_class("td","target",targets)]))]
 	return {doc:drawing,sources:sources,targets:targets}
     default:
 	var segments=mapix(bind(t.draw_segment,t),doc.segments)
@@ -670,7 +672,7 @@ Translator.prototype.draw_segment_given_target=function(s,target,i) {
     source.onclick=function() { t.edit_source(source,i); }
     var options=wrap_class("td","options",draw_options(s.options))
 
-    return wrap_class("tr","segment",[actions,source,options,target])
+    return node("tr",{"class":"segment",id:i},[actions,source,options,target])
 }
 
 function empty_document() {