Merge pull request #450 from darthleslin/leslin-patch-1

discuss page contents space fix
codeigniter4projects · Jun 8, 2024 · 22b7f25 · 22b7f25
2 parents 52f3af0 + ea63fb0
commit 22b7f25
Show file tree

Hide file tree

Showing 2 changed files with 40 additions and 14 deletions.
diff --git a/app/Views/discuss.php b/app/Views/discuss.php
@@ -28,7 +28,7 @@
         <div class="inner-page-text-box">
             <div class="inner-page-text-box-title">Forum</div>
             <p>
-                Our forum has been setup using MyBB, with five main categories: <br />
+                Our forum has been setup using MyBB (MyBulletinBoard), with five main categories:
             </p>
             <ul>
                 <li>General (news & discussion, lounge, events, and regional user groups)</li>
@@ -39,11 +39,10 @@
                 <li>External Resources (addins, jobs, learn more, and spotlight)</li>
             </ul>
             <p>
-                The forum is where you can ask for help or discuss issues you are having with framework.<br />
-                <a href="https://forum.codeigniter.com" class="buttons" target="_blank">Visit The Forum</a>
+              The forum is where you can ask for help or discuss issues you are having with framework.
             </p>
-            <br /><br />
-        </div><!--inner-page-text-box ende-->
+            <a href="https://forum.codeigniter.com" class="buttons cta-btn" target="_blank">Visit The Forum</a>
+        </div><!--inner-page-text-box ends here-->
 
         <div class="clr"></div>
 
@@ -56,10 +55,7 @@
                    class="link-primary" target="_blank">signup</a> for it :)
             </p>
 
-            <p>
-                <a href="https://join.slack.com/t/codeigniterchat/shared_invite/zt-rl30zw00-obL1Hr1q1ATvkzVkFp8S0Q" class="buttons" target="_blank">Slack</a>
-            </p>
-            <br /><br />
+            <a href="https://join.slack.com/t/codeigniterchat/shared_invite/zt-rl30zw00-obL1Hr1q1ATvkzVkFp8S0Q" class="buttons cta-btn" target="_blank">Visit Slack Channel</a>
         </div><!--inner-page-text-box ends here-->
 
         <div class="clr"></div>
@@ -76,11 +72,10 @@ class="link-primary" target="_blank">signup</a> for it :)
             <p>
                 GitHub issues are also used for tracking planned and approved enhancements, often tied in to specific releases.
             </p>
-            <p>
-                <a href="https://github.com/bcit-ci/CodeIgniter/" class="buttons" target="_blank">CodeIgniter 3</a>
-                <a href="https://github.com/codeigniter4/codeigniter4/" class="buttons" target="_blank">CodeIgniter 4</a>
-            </p>
-            <br /><br />
+
+            <a href="https://github.com/bcit-ci/CodeIgniter/" class="buttons cta-btn" target="_blank">CodeIgniter 3</a>
+            <a href="https://github.com/codeigniter4/codeigniter4/" class="buttons cta-btn" target="_blank">CodeIgniter 4</a>
+
         </div><!--inner-page-text-box ends here-->
 
         <div class="clr"></div>

diff --git a/public/assets/css/ci-theme.css b/public/assets/css/ci-theme.css
@@ -2124,3 +2124,34 @@ html.dark-theme #discuss-icon-holder {
 html.dark-theme #discuss-icon {
     fill: rgba(221, 72, 20, 0.9);
 }
+
+.inner-page-text-box {
+    padding: 32px;
+}
+
+.cta-btn {
+    margin: 0;
+    float: none;
+    display: inline-block;
+}
+
+.cta-btn:not(:first-of-type) {
+    margin-left: 1rem;
+}
+
+ul li {
+    margin-top: 1rem;
+    line-height: 1em;
+}
+
+.warning {
+    padding: 10px 32px;
+}
+
+p {
+    line-height: 1.5em;
+}
+
+.inner-page-text-box {
+    margin: 32px auto;
+}