Prevent PurgeCSS from deleting dynamically-added version picker CSS classes (#2260)

anoadragon453 · web-flow · commit 705240da72dd · 2025-12-05T15:19:24.000Z
diff --git a/assets/scss/_styles_project.scss b/assets/scss/_styles_project.scss
@@ -51,19 +51,24 @@ Custom SCSS for the Matrix spec
     color: $black;
   }
 
-  /* Make the version dropdown scroll if its too large */
+  /* Make the version dropdown scroll if it's too large */
   ul#version-selector {
     max-height: 80vh;
     overflow-y: auto;
   }
 
+  /* As these styles are only applied by JavaScript, PurgeCSS doesn't see them
+   * in the source code and removes them unless we explicitly tell it not to.
+   */
+  /* purgecss start ignore */
   ul#version-selector li.selected a {
     font-weight: bold;
   }
 
   ul#version-selector li.latest a {
     color: $secondary;
   }
+  /* purgecss end ignore */
 }
 
 /* Styles for the sidebar nav */
diff --git a/changelogs/internal/newsfragments/2260.clarification b/changelogs/internal/newsfragments/2260.clarification
@@ -0,0 +1 @@
+Add version picker in the navbar.

Original file line number	Diff line number	Diff line change
`@@ -51,19 +51,24 @@ Custom SCSS for the Matrix spec`
`51`	`51`	`color: $black;`
`52`	`52`	`}`
`53`	`53`
`54`		`- /* Make the version dropdown scroll if its too large */`
	`54`	`+ /* Make the version dropdown scroll if it's too large */`
`55`	`55`	`ul#version-selector {`
`56`	`56`	`max-height: 80vh;`
`57`	`57`	`overflow-y: auto;`
`58`	`58`	`}`
`59`	`59`
	`60`	`+ /* As these styles are only applied by JavaScript, PurgeCSS doesn't see them`
	`61`	`+ * in the source code and removes them unless we explicitly tell it not to.`
	`62`	`+ */`
	`63`	`+ /* purgecss start ignore */`
`60`	`64`	`ul#version-selector li.selected a {`
`61`	`65`	`font-weight: bold;`
`62`	`66`	`}`
`63`	`67`
`64`	`68`	`ul#version-selector li.latest a {`
`65`	`69`	`color: $secondary;`
`66`	`70`	`}`
	`71`	`+ /* purgecss end ignore */`
`67`	`72`	`}`
`68`	`73`
`69`	`74`	`/* Styles for the sidebar nav */`