Skip to content

Commit 3a6d5f7

Browse files
csharpfritzcsharpfritz
committed
Refactor layout and styles for improved readability and consistency; enhance Terms page content
1 parent 641f2b5 commit 3a6d5f7

4 files changed

Lines changed: 64 additions & 34 deletions

File tree

‎Web/Pages/Shared/_Layout.cshtml‎

Lines changed: 25 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,18 @@
11
@inject IConfiguration Configuration
22
<!DOCTYPE html>
33
<html lang="en">
4-
<head> <meta charset="utf-8" />
4+
<head>
5+
<meta charset="utf-8" />
56
<meta name="viewport" content="width=device-width, initial-scale=1.0" />
6-
<title>@ViewData["Title"] - Copilot That Jawn</title>
77
<meta name="description" content="@(ViewData["Description"] ?? "Where Philly innovation meets AI excellence. Master Microsoft Copilot and GitHub Copilot with our expert-curated tips.")" />
8+
<title>@ViewData["Title"] - Copilot That Jawn</title>
89

9-
<!-- XML Feeds -->
10+
<!-- Feeds and Sitemap -->
1011
<link rel="sitemap" type="application/xml" title="Sitemap" href="/sitemap.xml" />
11-
<link rel="alternate" type="application/rss+xml" title="Copilot That Jawn - Latest Tips" href="/feed.rss" />
12-
<script type="importmap"></script>
13-
<!-- Import Outfit font for brand -->
14-
<link href="https://fonts.googleapis.com/css2?family=Outfit:wght@800&display=swap" rel="stylesheet">
12+
<link rel="alternate" type="application/rss+xml" title="Latest Tips" href="/feed.rss" />
13+
14+
<!-- Fonts -->
15+
<link href="https://fonts.googleapis.com/css2?family=Outfit:wght@800&display=swap" rel="stylesheet" />
1516
<link rel="stylesheet" href="~/lib/bootstrap/dist/css/bootstrap.min.css" />
1617
<!-- Bootstrap Icons for theme toggle -->
1718
<link rel="stylesheet" href="https://cdn.jsdelivr.net/npm/[email protected]/font/bootstrap-icons.css" /> <!-- PrismJS CSS from CDN - will load theme based on user preference -->
@@ -36,22 +37,21 @@
3637
<div class="container"> <a class="navbar-brand d-flex align-items-center" asp-area="" asp-page="/Index">
3738
<img src="~/img/icon-100.webp" alt="Copilot That Jawn" class="me-2" style="height: 24px; width: auto;">
3839
<span class="brand-title">Copilot That Jawn</span>
39-
</a>
40-
<button class="navbar-toggler" type="button" data-bs-toggle="collapse" data-bs-target=".navbar-collapse" aria-controls="navbarSupportedContent"
40+
</a> <button class="navbar-toggler" type="button" data-bs-toggle="collapse"
41+
data-bs-target=".navbar-collapse" aria-controls="navbarSupportedContent"
4142
aria-expanded="false" aria-label="Toggle navigation">
4243
<span class="navbar-toggler-icon"></span>
43-
</button> <div class="navbar-collapse collapse d-sm-inline-flex justify-content-between">
44-
<ul class="navbar-nav">
45-
@* <li class="nav-item">
44+
</button>
45+
<div class="navbar-collapse collapse d-sm-inline-flex justify-content-between">
46+
<ul class="navbar-nav">
47+
<li class="nav-item">
4648
<a class="nav-link" asp-page="/Tips/Index">Tips</a>
4749
</li>
48-
4950
<li class="nav-item">
50-
<a class="nav-link" asp-area="" asp-page="/Privacy">Privacy</a>
51-
</li> *@
51+
<a class="nav-link" asp-page="/About">About</a>
52+
</li>
5253
</ul>
5354

54-
<!-- Theme toggle will be injected here by JavaScript -->
5555
<ul class="navbar-nav ms-auto" id="navbarTools">
5656
<li class="nav-item" id="themeToggleContainer"></li>
5757
</ul>
@@ -94,16 +94,22 @@
9494
</div>
9595
</div>
9696
</div>
97-
</footer><script src="~/lib/jquery/dist/jquery.min.js"></script>
97+
</footer> <!-- Core libraries -->
98+
<script src="~/lib/jquery/dist/jquery.min.js"></script>
9899
<script src="~/lib/bootstrap/dist/js/bootstrap.bundle.min.js"></script>
99-
<!-- PrismJS from CDN with comprehensive language support -->
100+
101+
<!-- Syntax highlighting -->
100102
<script src="https://cdnjs.cloudflare.com/ajax/libs/prism/1.29.0/prism.min.js"></script>
101103
<script src="https://cdnjs.cloudflare.com/ajax/libs/prism/1.29.0/components/prism-csharp.min.js"></script>
102104
<script src="https://cdnjs.cloudflare.com/ajax/libs/prism/1.29.0/components/prism-javascript.min.js"></script>
103105
<script src="https://cdnjs.cloudflare.com/ajax/libs/prism/1.29.0/components/prism-typescript.min.js"></script>
104106
<script src="https://cdnjs.cloudflare.com/ajax/libs/prism/1.29.0/components/prism-powershell.min.js"></script>
105107
<script src="https://cdnjs.cloudflare.com/ajax/libs/prism/1.29.0/components/prism-json.min.js"></script>
106-
<script src="https://cdnjs.cloudflare.com/ajax/libs/prism/1.29.0/components/prism-bash.min.js"></script> <script src="https://cdnjs.cloudflare.com/ajax/libs/prism/1.29.0/components/prism-yaml.min.js"></script> <script src="~/js/site.js" asp-append-version="true"></script>
108+
<script src="https://cdnjs.cloudflare.com/ajax/libs/prism/1.29.0/components/prism-bash.min.js"></script>
109+
<script src="https://cdnjs.cloudflare.com/ajax/libs/prism/1.29.0/components/prism-yaml.min.js"></script>
110+
111+
<!-- Site scripts -->
112+
<script src="~/js/site.js" asp-append-version="true"></script>
107113
<script src="~/js/theme-switcher.js" asp-append-version="true"></script>
108114
<script src="~/js/analytics.js" asp-append-version="true"></script>
109115
<script>

‎Web/Pages/Terms.cshtml‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -69,10 +69,9 @@
6969
<li>Allow 24-48 hours for maintainers to review</li>
7070
</ul>
7171
<p>For urgent content moderation inquiries: <a href="mailto:[email protected]">moderation@copilotthatjawn.com</a></p>
72-
</div>
73-
74-
<div class="terms-section mb-4">
72+
</div> <div class="terms-section mb-4">
7573
<h2 class="h4 mb-3">5. AI Tools and Services</h2>
74+
7675
<p>Information about AI tools and services is provided for educational purposes only:</p>
7776
<ul>
7877
<li>All AI-generated content must be reviewed for workplace appropriateness</li>
@@ -81,6 +80,7 @@
8180
<li>AI-generated code must comply with professional standards</li>
8281
<li>Users are responsible for ensuring AI outputs are work-appropriate</li>
8382
</ul>
83+
8484
<p class="mt-3">When using our AI tools:</p>
8585
<ul>
8686
<li>Maintain professional context in all prompts and interactions</li>

‎Web/wwwroot/css/layout.css‎

Lines changed: 23 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,31 @@
11
/* Layout and Footer Styles */
2-
html, body {
2+
:root {
3+
/* Z-index hierarchy */
4+
--z-index-dropdown: 1050;
5+
--z-index-sticky: 1020;
6+
--z-index-code-copy: 10;
7+
}
8+
9+
html {
310
height: 100%;
4-
margin: 0;
11+
font-size: 14px;
12+
position: relative;
13+
}
14+
15+
@media (min-width: 768px) {
16+
html {
17+
font-size: 16px;
18+
}
519
}
620

721
body {
822
display: flex;
923
flex-direction: column;
1024
min-height: 100vh;
25+
margin: 0;
26+
background-color: var(--bg-primary);
27+
color: var(--text-primary);
28+
transition: background-color 0.3s ease, color 0.3s ease;
1129
}
1230

1331
.container {
@@ -22,7 +40,7 @@ body {
2240
main {
2341
flex: 1 0 auto;
2442
width: 100%;
25-
min-height: calc(100vh - 300px); /* Account for header and footer */
43+
padding-bottom: 4rem;
2644
}
2745

2846
.footer {
@@ -33,4 +51,6 @@ main {
3351
border-top: 1px solid var(--border-color);
3452
padding: 2rem 0;
3553
margin-top: auto;
54+
position: static;
55+
line-height: 1.5;
3656
}

‎Web/wwwroot/css/site.css‎

Lines changed: 13 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,16 @@
1-
/* CSS Custom Properties for Theme Management */
2-
:root { /* Purple primary color scheme */
1+
/* Theme Management */
2+
:root {
3+
/* Primary color scheme */
34
--primary-color: #6200EE;
45
--primary-hover: #5600E8;
56
--primary-light: #7F39FB;
67
--primary-dark: #5000C9;
78

8-
/* Override Bootstrap primary */ --bs-primary: var(--primary-color);
9+
/* Bootstrap overrides */
10+
--bs-primary: var(--primary-color);
911
--bs-primary-rgb: 98, 0, 238;
1012

11-
/* Light theme colors (default) */
13+
/* Light theme colors */
1214
--bg-primary: #ffffff;
1315
--bg-secondary: #f8f9fa;
1416
--bg-tertiary: #e9ecef;
@@ -402,16 +404,18 @@ pre[class*="language-"] {
402404
transition: border-color 0.3s ease;
403405
}
404406

405-
/* Custom font for code blocks */
407+
/* Code styling with modern fonts */
406408
pre[class*="language-"] code,
407-
code[class*="language-"] {
408-
font-family: 'Fira Code', 'JetBrains Mono', 'Source Code Pro', 'Cascadia Code', 'SF Mono', Consolas, 'Liberation Mono', 'Menlo', Monaco, 'Courier New', monospace;
409+
code[class*="language-"],
410+
:not(pre) > code {
411+
font-family: 'Fira Code', 'JetBrains Mono', 'Source Code Pro', 'Cascadia Code', 'SF Mono', Consolas, monospace;
409412
font-size: 0.875rem;
410413
font-weight: 500;
411414
line-height: 1.6;
412415
letter-spacing: 0.025em;
413-
white-space: pre;
414-
font-feature-settings: "liga" 1, "calt" 1; /* Enable ligatures for supported fonts */
416+
font-feature-settings: "liga" 1, "calt" 1;
417+
-webkit-font-smoothing: antialiased;
418+
-moz-osx-font-smoothing: grayscale;
415419
}
416420

417421
/* Theme-specific adjustments - minimal overrides */

0 commit comments

Comments
 (0)