Added a 5px margin top to read only elements, so they line up better …

…with labels (which have a 5px margin top to account for vertical centering with input fields)
keianrad · Dec 23, 2016 · ae73774 · ae73774
1 parent 1963cd9
commit ae73774
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/components/com_fabrik/models/element.php b/components/com_fabrik/models/element.php
@@ -2378,7 +2378,7 @@ public function preRenderElement($data, $repeatCounter = 0)
 			$v = $this->getROElement($data, $repeatCounter);
 			//$v = $v == '' ? '&nbsp;' : $v;
 
-			return '<div id="' . $htmlId . '">' . $v . '</div>';
+			return '<div class="fabrikElementReadOnly" id="' . $htmlId . '">' . $v . '</div>';
 		}
 	}
 

diff --git a/media/com_fabrik/css/fabrik.css b/media/com_fabrik/css/fabrik.css
@@ -563,6 +563,11 @@ td.repeat-reduce ul.fabrikRepeatData li {
 /************  Misc. Workarounds *************************/
 /*********************************************************/
 
+/** add a top margin for read only elements **/
+.fabrikElementReadOnly {
+	margin-top: 5px;
+}
+
 /** Workaround for Gantry's date class stomping on our date element class **/
 .fabrikGroup ul li.date {
 	font-weight: normal;