GSA / notifications-admin

The UI of Notify.gov
https://notify.gov
Other
11 stars 2 forks source link

Added link to new guidance page that enables navigating #2078

Closed alexjanousekGSA closed 1 week ago

alexjanousekGSA commented 3 weeks ago

Description

Screenshot

image
alexjanousekGSA commented 3 weeks ago

Looks great!

Just one change...in best-practices.html, the links within card_contents needs to be changed to ./best-practices

  {% set card_contents = [
  {
  "svg_src": "goal",
  "card_heading": "Establish clear goals",
  "p_text": "Start with a singular purpose. Make explicit what you want to achieve.",
  "link": "/best-practices/clear-goals"
  },
  {
  "svg_src": "compliant",
  "card_heading": "Follow rules & regulations",
  "p_text": "Understand what is required when texting the public.",
  "link": "/best-practices/rules-and-regulations"
  },
  {
  "svg_src": "trust",
  "card_heading": "Establish trust",
  "p_text": "Help your audience anticipate and welcome your texts.",
  "link": "/best-practices/establish-trust"
  },
  {
  "svg_src": "runner",
  "card_heading": "Write texts that provoke action",
  "p_text": "Help your audience know what to do with the information you send.",
  "link": "/best-practices/write-for-action"
  },
  {
  "svg_src": "language",
  "card_heading": "Send texts in multiple languages",
  "p_text": "What to know as you plan translated texts.",
  "link": "/best-practices/multiple-languages"
  },
  {
  "svg_src": "chart",
  "card_heading": "Measure performance with benchmarking",
  "p_text": "Learn how effective your texting program can be.",
  "link": "/best-practices/benchmark-performance"
  }
  ] %}

Awesome catch. Updated and tested!

jonathanbobel commented 2 weeks ago

The top nav item in the main nav should be "Guides" instead of "Best practices" - everything else stays the same