frontend icon indicating copy to clipboard operation
frontend copied to clipboard

Top nav improvements

Open mxdvl opened this issue 2 years ago • 1 comments

Here’s a series of step I think we should take to follow-up on https://github.com/guardian/dotcom-rendering/issues/6436 :

  • Remove the headerTopNav switch
  • Refactor the headerContent logic to only have two possible paths
  • Reduce the header files to two:
    • rename headerTopNav.scala.html to header.scala.html
      • adjust this file so it reuses some of the patterns originally defined in header.scala.html
    • rename header.scala.html to headerSlim.scala.html and remove all the non-slim code paths

mxdvl avatar Nov 28 '22 12:11 mxdvl

related to #9341 ?

shesah avatar Nov 16 '23 15:11 shesah