From b88d23a270b371288f73dda1f7de8fa3afc11b97 Mon Sep 17 00:00:00 2001
From: Edward Betts <edward@4angle.com>
Date: Wed, 1 Apr 2026 15:33:07 +0100
Subject: [PATCH] Use destination-specific result page metadata

---
 templates/base.html    | 10 +++++++++-
 templates/results.html |  5 +++++
 tests/test_app.py      | 17 +++++++++++++++++
 3 files changed, 31 insertions(+), 1 deletion(-)
diff --git a/templates/base.html b/templates/base.html
index 47c41b5..aa65c78 100644
--- a/templates/base.html
+++ b/templates/base.html
@@ -3,7 +3,15 @@
 <head>
   <meta charset="UTF-8">
   <meta name="viewport" content="width=device-width, initial-scale=1">
-  <title>Bristol to Europe via Eurostar</title>
+  <title>{% block title %}Bristol to Europe via Eurostar{% endblock %}</title>
+  <meta property="og:type" content="website">
+  <meta property="og:site_name" content="Bristol to Europe via Eurostar">
+  <meta property="og:title" content="{% block og_title %}Bristol to Europe via Eurostar{% endblock %}">
+  <meta property="og:description" content="{% block og_description %}Find Bristol Temple Meads to Europe itineraries via Paddington, St Pancras, and Eurostar.{% endblock %}">
+  <meta property="og:url" content="{{ request.url }}">
+  <meta name="twitter:card" content="summary">
+  <meta name="twitter:title" content="{% block twitter_title %}Bristol to Europe via Eurostar{% endblock %}">
+  <meta name="twitter:description" content="{% block twitter_description %}Find Bristol Temple Meads to Europe itineraries via Paddington, St Pancras, and Eurostar.{% endblock %}">
   <link rel="icon" href="{{ url_for('static', filename='favicon.svg') }}" type="image/svg+xml">
   <style>
     *, *::before, *::after { box-sizing: border-box; }
diff --git a/templates/results.html b/templates/results.html
index 2e39bba..3e38faa 100644
--- a/templates/results.html
+++ b/templates/results.html
@@ -1,4 +1,9 @@
 {% extends "base.html" %}
+{% block title %}Bristol to {{ destination }} via Eurostar{% endblock %}
+{% block og_title %}Bristol to {{ destination }} via Eurostar{% endblock %}
+{% block og_description %}Train options from Bristol Temple Meads to {{ destination }} on {{ travel_date_display }} via Paddington, St Pancras, and Eurostar.{% endblock %}
+{% block twitter_title %}Bristol to {{ destination }} via Eurostar{% endblock %}
+{% block twitter_description %}Train options from Bristol Temple Meads to {{ destination }} on {{ travel_date_display }} via Paddington, St Pancras, and Eurostar.{% endblock %}
 {% block content %}
 
 <p style="margin-bottom:1rem">
diff --git a/tests/test_app.py b/tests/test_app.py
index c8d9dce..0292dad 100644
--- a/tests/test_app.py
+++ b/tests/test_app.py
@@ -72,3 +72,20 @@ def test_results_shows_same_day_destination_switcher(monkeypatch):
     assert '/results/brussels/2026-04-10?min_connection=60&amp;max_connection=120' in html
     assert '/results/rotterdam/2026-04-10?min_connection=60&amp;max_connection=120' in html
     assert 'ES 9014' in html
+
+
+def test_results_title_and_social_meta_include_destination(monkeypatch):
+    _stub_data(monkeypatch)
+    client = _client()
+
+    resp = client.get('/results/lille/2026-04-10?min_connection=60&max_connection=120')
+    html = resp.get_data(as_text=True)
+
+    assert resp.status_code == 200
+    assert '<title>Bristol to Lille Europe via Eurostar</title>' in html
+    assert '<meta property="og:title" content="Bristol to Lille Europe via Eurostar">' in html
+    assert (
+        '<meta property="og:description" content="Train options from Bristol Temple Meads '
+        'to Lille Europe on Friday 10 April 2026 via Paddington, St Pancras, and Eurostar.">'
+    ) in html
+    assert '<meta property="og:url" content="http://localhost/results/lille/2026-04-10?min_connection=60&amp;max_connection=120">' in html
