<div dir="ltr">Hi Thomas,<div><br></div><div>thanks for the hint.</div><div>There are 2 problems.</div><div>1. there should of course be only one style entry for this</div><div>2. the 2 way diff should not be put in another panel</div><div><br></div><div>I'll push 2 more changesets to fix this.</div><div><br></div><div>cheers</div><div>Dominik<br><br><div class="gmail_quote"><div dir="ltr">Thomas De Schampheleire <<a href="mailto:patrickdepinguin@gmail.com">patrickdepinguin@gmail.com</a>> schrieb am Mo., 29. Jan. 2018 um 13:27 Uhr:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Dominik,<br>
<br>
After upgrading our Kallithea to something based on almost latest<br>
default branch, we noticed an issue with side-by-side diff in some<br>
browsers, e.g.<br>
- Firefox 52 (Windows)<br>
- Chrome 63 (Windows)<br>
<br>
I don't see the issue on:<br>
- Firefox 57 (Linux)<br>
<br>
The issue is that the right pane does not seem to contain any code,<br>
and as such there is no diff.<br>
Please see screenshots. Second screenshot is same page as first but<br>
scrolled down a bit.<br>
<br>
I bisected the issue down to the commit below.<br>
I confirmed that by removing that block on the latest default, two-way<br>
diff works fine again. However, I don't know what else I broke now.<br>
<br>
Best regards,<br>
Thomas<br>
<br>
<br>
# HG changeset patch<br>
# User domruf <<a href="mailto:dominikruf@gmail.com" target="_blank">dominikruf@gmail.com</a>><br>
# Date 1509208744 -7200<br>
#      Sat Oct 28 18:39:04 2017 +0200<br>
# Node ID eee7674cef4126d3ad15754992ac1eb20f4587d8<br>
# Parent  5a5398a5bf831d33808e0d6261a0516b27366cd2<br>
less: prevent elements from overflowing panels<br>
<br>
diff --git a/kallithea/public/css/style.css b/kallithea/public/css/style.css<br>
--- a/kallithea/public/css/style.css<br>
+++ b/kallithea/public/css/style.css<br>
@@ -102,6 +102,12 @@ nav.navbar #logo > .navbar-brand > img {<br>
   margin-top: -9px;<br>
   margin-right: 5px;<br>
 }<br>
+/* every direct child of a panel, that is not .panel-heading, should auto<br>
+ * overflow to prevent overflowing of elements like text boxes and tables */<br>
+.panel > :not(.panel-heading) {<br>
+  overflow-x: auto;<br>
+  min-height: 0.01%;<br>
+}<br>
 /* language bars (summary page) */<br>
 #lang_stats .progress-bar {<br>
   min-width: 15px;<br>
diff --git a/kallithea/public/less/style.less b/kallithea/public/less/style.less<br>
--- a/kallithea/public/less/style.less<br>
+++ b/kallithea/public/less/style.less<br>
@@ -111,6 +111,13 @@ nav.navbar #logo > .navbar-brand > img {<br>
   margin-right: 5px;<br>
 }<br>
<br>
+/* every direct child of a panel, that is not .panel-heading, should auto<br>
+ * overflow to prevent overflowing of elements like text boxes and tables */<br>
+.panel > :not(.panel-heading){<br>
+  overflow-x: auto;<br>
+  min-height: 0.01%;<br>
+}<br>
+<br>
 /* language bars (summary page) */<br>
 #lang_stats {<br>
   .progress-bar {<br>
</blockquote></div></div></div>