Skip to content

Comments

Fix markdown rendering in blog posts and replace nav logo with bicycle icon#14

Merged
miacycle merged 2 commits intomainfrom
copilot/fix-blog-post-markdown
Feb 19, 2026
Merged

Fix markdown rendering in blog posts and replace nav logo with bicycle icon#14
miacycle merged 2 commits intomainfrom
copilot/fix-blog-post-markdown

Conversation

Copy link
Contributor

Copilot AI commented Feb 19, 2026

Blog post markdown was not visually styled (prose plugin missing), and the nav logo displayed a plain "MG" text instead of a thematic bicycle icon.

Changes

  • tailwind.config.ts — Added @tailwindcss/typography to the plugins array. The prose class was already applied in BlogPost.tsx but had no effect without the plugin.

    plugins: [require("tailwindcss-animate"), require("@tailwindcss/typography")],
  • Navigation.tsx — Swapped the "MG" text for a Bike icon from lucide-react (already a dependency).

    <Link to="/" className="text-primary hover:text-primary/80 transition-colors">
      <Bike className="h-7 w-7" />
    </Link>

Screenshots

Bicycle icon in navigation + homepage:
Homepage

Rendered markdown in blog post (bold, lists, links):
Blog post


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: miacycle <184569369+miacycle@users.noreply.github.com>
Copilot AI changed the title [WIP] Ensure proper rendering of blog post markdown Fix markdown rendering in blog posts and replace nav logo with bicycle icon Feb 19, 2026
Copilot AI requested a review from miacycle February 19, 2026 14:59
@miacycle miacycle marked this pull request as ready for review February 19, 2026 15:05
Copilot AI review requested due to automatic review settings February 19, 2026 15:05
@miacycle miacycle merged commit 8449221 into main Feb 19, 2026
5 checks passed
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes markdown styling in blog posts and replaces the plain "MG" text logo with a thematic bicycle icon in the navigation bar. The changes enable proper visual rendering of markdown content using Tailwind's typography plugin and provide a more visually distinctive branding element.

Changes:

  • Added @tailwindcss/typography plugin to enable prose styling for markdown content in blog posts
  • Replaced text-based "MG" logo with a Bike icon from lucide-react in the navigation component
  • Updated package-lock.json to reflect the new typography plugin dependency

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

File Description
tailwind.config.ts Added @tailwindcss/typography plugin to enable prose class styling
src/components/Navigation.tsx Replaced "MG" text logo with Bike icon and updated import statement
package-lock.json Added @tailwindcss/typography package and its dependencies as devDependencies

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

<div className="flex items-center justify-between">
<Link to="/" className="text-xl font-semibold text-primary">
MG
<Link to="/" className="text-primary hover:text-primary/80 transition-colors">
Copy link

Copilot AI Feb 19, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Bike icon link is missing an aria-label attribute for accessibility. Screen reader users won't have clear information about where this link navigates to. Add an aria-label describing the link's purpose (e.g., "Home" or "Go to homepage").

Suggested change
<Link to="/" className="text-primary hover:text-primary/80 transition-colors">
<Link
to="/"
className="text-primary hover:text-primary/80 transition-colors"
aria-label="Go to homepage"
>

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants