From c9bd2f2751306979f1523dab442e1e5b2bb903c7 Mon Sep 17 00:00:00 2001
From: Dominik Wombacher <dominik@wombacher.cc>
Date: Jan 13 2023 14:31:42 +0000
Subject: fix(theme): long words in source nav break layout


The 'srcfpo' theme adds additional information to the commits side navigation, which might contain long words and then break the layout.
In that case the content appeared below the navigation and not side-by-side with the commits overview or commit details.
To fix this, a text break will be enforced in the relevant div of the 'srcfpo' theme.
Also the column width in global 'commit.html' aligned with other template files by setting to 'col-2'.

Fixes: https://pagure.io/pagure/issue/5353

---

diff --git a/pagure/templates/commit.html b/pagure/templates/commit.html
index 73408ec..4238e0d 100644
--- a/pagure/templates/commit.html
+++ b/pagure/templates/commit.html
@@ -20,7 +20,7 @@
 {# we recognize non-executable file, executable file and symlink #}
 {% set expected_modes = [33188, 33261, 40960] %}
 <div class="row">
-  <div class="col">
+  <div class="col-2">
       {% block overviewtabs %}{{ super() }}{% endblock %}
   </div>
   <div class="col-10">
diff --git a/pagure/themes/srcfpo/static/theme.css b/pagure/themes/srcfpo/static/theme.css
index 1831daf..45a5600 100644
--- a/pagure/themes/srcfpo/static/theme.css
+++ b/pagure/themes/srcfpo/static/theme.css
@@ -13,4 +13,9 @@
 
 .footer a.notblue{
   opacity:0.7;
-}
\ No newline at end of file
+}
+
+.enforce-text-break {
+  word-wrap: break-word !important;
+  word-break: break-all !important;
+}
diff --git a/pagure/themes/srcfpo/templates/repo_master_sidebar.html b/pagure/themes/srcfpo/templates/repo_master_sidebar.html
index ee0524a..b3120e9 100644
--- a/pagure/themes/srcfpo/templates/repo_master_sidebar.html
+++ b/pagure/themes/srcfpo/templates/repo_master_sidebar.html
@@ -1,4 +1,4 @@
-<nav class="nav nav-tabs nav-sidetabs flex-column">
+<nav class="nav nav-tabs nav-sidetabs flex-column flex-nowrap">
   <a class=
       "nav-link nowrap
       {%if select == 'overview' %} active{% endif %}"
@@ -66,7 +66,7 @@
   </a>
 
   {% if 'distgit_ns' in g.main_app.blueprints and not repo.is_fork and repo.namespace != 'tests'%}
-  <div class="col-xs-2 line-height-1"></div>
+  <div class="line-height-1"></div>
   <h6>Monitoring status:</h6>
   <div class="btn-group">
     {% if g.authenticated %}
@@ -158,9 +158,9 @@
     {% endif %}
   </div>
 
-  <div class="col-xs-2 line-height-1"></div>
-  <div id="orphan-section" class="pt-3">
-      <div class="col-xs-2 line-height-1"></div>
+  <div class="line-height-1"></div>
+  <div id="orphan-section" class="pt-3 enforce-text-break">
+      <div class="line-height-1"></div>
       {% if repo.user.user == "orphan" %}
       <p>Orphaned for: {{ repo.orphan_reason.reason }}
       {% if repo.orphan_reason.reason_info %}
@@ -194,7 +194,7 @@
   </div>
 
   <div class="pt-3">
-    <div class="col-xs-2 line-height-1">
+    <div class="line-height-1">
     <h6>Bugzilla Assignee:</h6>
       <dl>
         <dt>Fedora: </dt>