fix bugs in classic theme's templates

rendercv/themes/classic/EducationEntry.j2.tex

@@ -1,3 +1,11 @@
+((* if section_title in design.show_time_span_in *))
+    ((* set date_and_location_strings = [entry.location, entry.date_string, entry.time_span_string] *))
+((* else *))
+    ((* set date_and_location_strings = [entry.location, entry.date_string] *))
+((* endif *))
+((* if not is_first_entry *))
+\vspace{<<design.margins.entry_area.vertical_between>>}
+((* endif *))
 ((# \begin{tabularx}{⟨width⟩}[⟨pos⟩]{⟨preamble⟩} #))
     ((# width: \textwidth #))
     ((# preamble: first column, second column, third column #))
@@ -5,12 +13,12 @@
         ((# second column: K{<<design.margins.entry_area.left_and_right>>}; variable width, justified column #))
         ((# third column: R{<<design.date_and_location_width>>}; constant widthm ragged right column #))
 \begin{tabularx}{
-        \textwidth-<<design.margins.entry_area.left_and_right|divide_length_by(0.5)>>-0.13cm
+    \textwidth-<<design.margins.entry_area.left_and_right|divide_length_by(0.5)>>-0.13cm
-    }{
+}{
-        L{0.85cm}
+    L{0.85cm}
-        K{<<design.margins.entry_area.left_and_right>>}
+    K{<<design.margins.entry_area.left_and_right>>}
-        R{<<design.date_and_location_width>>}
+    R{<<design.date_and_location_width>>}
-    }
+}
     \textbf{<<entry.study_type>>}
     &
     \textbf{<<entry.institution>>}, <<entry.area>>
@@ -22,15 +30,15 @@
     \begin{highlights}
     ((* endif *))
         \item <<item>>
-
     ((* if loop.last *))
     \end{highlights}
     ((* endif *))
 ((* endfor *))
     &
-((*set date_and_location_strings = [entry.location, entry.date_string, entry.time_span_string]*))
 ((* for item in date_and_location_strings *))
     <<item>>
+    ((* if not loop.last *))
 
+    ((* endif *))
 ((* endfor *))
 \end{tabularx}

rendercv/themes/classic/ExperienceEntry.j2.tex

@@ -1,14 +1,22 @@
+((* if section_title in design.show_time_span_in *))
+    ((* set date_and_location_strings = [entry.location, entry.date_string, entry.time_span_string] *))
+((* else *))
+    ((* set date_and_location_strings = [entry.location, entry.date_string] *))
+((* endif *))
+((* if not is_first_entry *))
+\vspace{<<design.margins.entry_area.vertical_between>>}
+((* endif *))
 ((# \begin{tabularx}{⟨width⟩}[⟨pos⟩]{⟨preamble⟩} #))
     ((# width: \textwidth #))
     ((# preamble: first column, second column #))
         ((# first column:: K{<<design.margins.entry_area.left_and_right>>}; variable width, justified column #))
         ((# second column: R{<<design.date_and_location_width>>}; constant width ragged right column #))
 \begin{tabularx}{
-        \textwidth-<<design.margins.entry_area.left_and_right|divide_length_by(0.5)>>-0.13cm
+    \textwidth-<<design.margins.entry_area.left_and_right|divide_length_by(0.5)>>-0.13cm
-    }{
+}{
-        K{<<design.margins.entry_area.left_and_right>>}
+    K{<<design.margins.entry_area.left_and_right>>}
-        R{<<design.date_and_location_width>>}
+    R{<<design.date_and_location_width>>}
-    }
+}
     \textbf{<<entry.company>>}, <<entry.position>>
 
     \vspace{<<design.margins.highlights_area.top>>}
@@ -18,15 +26,15 @@
     \begin{highlights}
     ((* endif *))
         \item <<item>>
-
     ((* if loop.last *))
     \end{highlights}
     ((* endif *))
 ((* endfor *))
     &
-((*set date_and_location_strings = [entry.location, entry.date_string, entry.time_span_string]*))
 ((* for item in date_and_location_strings *))
     <<item>>
+    ((* if not loop.last *))
 
+    ((* endif *))
 ((* endfor *))
 \end{tabularx}

rendercv/themes/classic/Header.j2.tex

@@ -1,3 +1,7 @@
+((* if design.show_last_updated_date *))
+    \placelastupdatedtext
+((* endif *))
+
 \begin{header}
     \fontsize{<<design.header_font_size>>}{<<design.header_font_size>>}
     \textbf{<<cv.name>>}
@@ -34,7 +38,6 @@
     \mbox{\hrefWithoutArrow{network.url}{{\small<<icon_dictionary[network.network]>>}\hspace{0.13cm}<<network.username>>}}
     \hspace{0.5cm}
 ((* endfor *))
-
 \end{header}
 
 \vspace{<<design.margins.header.bottom>>}

rendercv/themes/classic/NormalEntry.j2.tex

@@ -1,25 +1,34 @@
-((*set date_and_location_strings = [entry.location, entry.date_string, entry.time_span_string]*))
+((* if section_title in design.show_time_span_in *))
-((* if date_and_location_strings == ["", "", ""]*))
+    ((* set date_and_location_strings = [entry.location, entry.date_string, entry.time_span_string] *))
+((* else *))
+    ((* set date_and_location_strings = [entry.location, entry.date_string] *))
+((* endif *))
+((* if not is_first_entry *))
+\vspace{<<design.margins.entry_area.vertical_between>>}
+((* endif *))
+((* if date_and_location_strings == ["", "", ""] or date_and_location_strings == ["", ""] *))
+((# \begin{tabularx}{⟨width⟩}[⟨pos⟩]{⟨preamble⟩} #))
+    ((# width: \textwidth #))
+    ((# preamble: first column #))
+        ((# first column:: K{<<design.margins.entry_area.left_and_right>>}; variable width, justified column #))
 \begin{tabularx}{
     \textwidth-<<design.margins.entry_area.left_and_right|divide_length_by(0.5)>>-0.13cm
 }{
     K{<<design.margins.entry_area.left_and_right>>}
 }
-\textbf{<<entry.name>>}
+    \textbf{<<entry.name>>}
 
-\vspace{<<design.margins.highlights_area.top>>}
+    \vspace{<<design.margins.highlights_area.top>>}
 
     ((* for item in entry.highlights *))
         ((* if loop.first *))
-\begin{highlights}
+    \begin{highlights}
         ((* endif *))
-    \item <<item>>
+        \item <<item>>
-
         ((* if loop.last *))
-\end{highlights}
+    \end{highlights}
         ((* endif *))
     ((* endfor *))
-
 \end{tabularx}
 ((* else *))
 ((# \begin{tabularx}{⟨width⟩}[⟨pos⟩]{⟨preamble⟩} #))
@@ -33,24 +42,25 @@
     K{<<design.margins.entry_area.left_and_right>>}
     R{<<design.date_and_location_width>>}
 }
-\textbf{<<entry.name>>}
+    \textbf{<<entry.name>>}
 
-\vspace{<<design.margins.highlights_area.top>>}
+    \vspace{<<design.margins.highlights_area.top>>}
 
     ((* for item in entry.highlights *))
         ((* if loop.first *))
-\begin{highlights}
+    \begin{highlights}
         ((* endif *))
-    \item <<item>>
+        \item <<item>>
-
         ((* if loop.last *))
-\end{highlights}
+    \end{highlights}
         ((* endif *))
     ((* endfor *))
-&
+    &
     ((* for item in date_and_location_strings *))
-<<item>>
+    <<item>>
+        ((* if not loop.last *))
 
+        ((* endif *))
     ((* endfor *))
 \end{tabularx}
 ((* endif *))

rendercv/themes/classic/OneLineEntry.j2.tex

@@ -1,3 +1,6 @@
+((* if not is_first_entry *))
+\vspace{<<design.margins.entry_area.vertical_between>>}
+((* endif *))
 \begingroup((* if design.text_alignment == "left-aligned" *))\raggedright((* endif *))
 \leftskip=<<design.margins.entry_area.left_and_right>>
 \advance\csname @rightskip\endcsname <<design.margins.entry_area.left_and_right>>

rendercv/themes/classic/Preamble.j2.tex

@@ -124,3 +124,8 @@
 \let\hrefWithoutArrow\href
  % new command for external links:
 \renewcommand{\href}[2]{\hrefWithoutArrow{#1}{#2 \raisebox{.15ex}{\footnotesize \faExternalLink*}}}
+
+\let\originalTabularx\tabularx
+\let\originalEndTabularx\endtabularx
+
+\renewenvironment{tabularx}{\bgroup\centering\originalTabularx}{\originalEndTabularx\par\egroup}

rendercv/themes/classic/PublicationEntry.j2.tex

@@ -1,3 +1,6 @@
+((* if not is_first_entry *))
+\vspace{<<design.margins.entry_area.vertical_between>>}
+((* endif *))
 ((# \begin{tabularx}{⟨width⟩}[⟨pos⟩]{⟨preamble⟩} #))
     ((# width: \textwidth #))
     ((# preamble: first column, second column #))
@@ -15,5 +18,4 @@
     \href{<<entry.doi_url>>}{<<entry.doi>>} (<<entry.journal>>)
     &
     <<entry.date_string>>
-
 \end{tabularx}

rendercv/themes/classic/TextEntry.j2.tex

@@ -1 +1,15 @@
+((* if not is_first_entry *))
+\vspace{<<design.margins.entry_area.vertical_between>>}
+((* endif *))
+{
+((* if design.text_alignment == "left-aligned" *))
+\raggedright
+((* endif *))
+\setlength{\leftskip}{<<design.margins.entry_area.left_and_right>>}
+\setlength{\rightskip}{<<design.margins.entry_area.left_and_right>>}
+
 <<entry>>
+
+\setlength{\leftskip}{0cm}
+\setlength{\rightskip}{0cm}
+}