feat(branding): add logo/favicon, rename to Librarian, widen layout#9
Merged
Conversation
Wire docs/icon.svg into the app as both favicon and inline logo:
served under static/ via a committed symlink in dev and a real
copy baked in at Docker build time. Add a reusable _brand.html
partial (logo + wordmark) and include it across all pages.
Rename the brand from "TKC Library" to "Librarian" in titles,
headings, and the wordmark. The physical-location reference in
the about page ("TKC Library at Kirby Street") is left as-is.
Widen the page container from 720px to 1080px so the book grid
has more room.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
This PR updates the app’s branding by introducing a new SVG logo/favicon, centralizing the brand lockup into a reusable template partial, renaming “TKC Library” to “Librarian” across titles/headings, and widening the main container to better fit the book grid.
Changes:
- Add
docs/icon.svgand wire it in as/static/brand/logo.svgfor favicon + logo usage (with a Docker build step to materialize the file). - Introduce
app/templates/_brand.htmland replace hardcoded brand text with the shared partial across most templates. - Update default/page titles to “Librarian” and widen
.containerfrom720pxto1080px.
Reviewed changes
Copilot reviewed 12 out of 14 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
docs/icon.svg |
Adds the new SVG brand mark source asset. |
Dockerfile |
Ensures the runtime image contains a real app/static/brand/logo.svg copy of the SVG. |
app/templates/base.html |
Updates default title, adds favicon links, widens container, and updates brand CSS. |
app/templates/_brand.html |
New shared brand lockup partial (logo + wordmark). |
app/library/templates/library/index.html |
Updates title/heading and swaps in the shared brand partial in the header. |
app/library/templates/library/dashboard.html |
Updates title and swaps in the shared brand partial. |
app/library/templates/library/add_book.html |
Updates title and swaps in the shared brand partial in the header. |
app/borrower/templates/borrower/dashboard.html |
Updates title to “Librarian”. |
app/borrower/templates/borrower/borrow_history.html |
Updates title to “Librarian”. |
app/admin/templates/admin/users.html |
Updates title and swaps in the shared brand partial. |
app/admin/templates/admin/dashboard.html |
Updates title and swaps in the shared brand partial. |
app/admin/templates/admin/borrowers.html |
Updates title and swaps in the shared brand partial. |
app/about/templates/about/about.html |
Updates title and swaps in the shared brand partial. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+43
to
+50
| .brand { | ||
| color: var(--brand); | ||
| font-weight: 700; | ||
| font-size: 1.1rem; | ||
| display: inline-flex; | ||
| align-items: center; | ||
| gap: 0.5rem; | ||
| } |
| {% extends "base.html" %} | ||
|
|
||
| {% block title %}Borrower Dashboard - TKC Library{% endblock %} | ||
| {% block title %}Borrower Dashboard - Librarian{% endblock %} |
| {% extends "base.html" %} | ||
|
|
||
| {% block title %}Borrow History - TKC Library{% endblock %} | ||
| {% block title %}Borrow History - Librarian{% endblock %} |
Comment on lines
+4
to
+7
| <div class="brand"> | ||
| <img class="brand-logo" src="{{ url_for('static', filename='brand/logo.svg') }}" alt="Librarian logo" /> | ||
| <span>Librarian</span> | ||
| </div> |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
docs/icon.svginto the app as both the favicon and an inline logo, served understatic/— a committed symlink in dev (live editing) and a real copy baked in at Docker build time (self-contained image)._brand.htmlpartial (logo + wordmark) and includes it across all pages, replacing the hardcoded brand text.720px→1080pxso the book grid has more room.Notes
app/static/brand/logo.svgis committed as a symlink (mode 120000) →docs/icon.svg. The Dockerfile replaces it with a real copy at build time so the runtime image doesn't depend on symlink resolution.Verification
Librarianbrand; Kirby Street prose preserved.GET /static/brand/logo.svgreturns200withContent-Type: image/svg+xml, byte-identical todocs/icon.svg.🤖 Generated with Claude Code