Actually fix sidebar tags with proper logic (forem#21289)

benhalpern · web-flow · commit e2b7d28399be · 2024-09-18T15:26:51.000-03:00
diff --git a/app/javascript/packs/homePage.jsx b/app/javascript/packs/homePage.jsx
@@ -88,9 +88,8 @@ function renderSidebar() {
 }
 
 const feedTimeFrame = frontPageFeedPathNames.get(window.location.pathname);
-const currentPage = document.getElementById('page-content')?.dataset.currentPage;
 
-if (currentPage === 'stories-index' && !document.getElementById('featured-story-marker')) {
+if (document.getElementById('sidebar-nav-followed-tags')) {
   const waitingForDataLoad = setInterval(() => {
     const { user = null, userStatus } = document.body.dataset;
     if (userStatus === 'logged-out') {

Original file line number	Diff line number	Diff line change
`@@ -88,9 +88,8 @@ function renderSidebar() {`
`88`	`88`	`}`
`89`	`89`
`90`	`90`	`const feedTimeFrame = frontPageFeedPathNames.get(window.location.pathname);`
`91`		`-const currentPage = document.getElementById('page-content')?.dataset.currentPage;`
`92`	`91`
`93`		`-if (currentPage === 'stories-index' && !document.getElementById('featured-story-marker')) {`
	`92`	`+if (document.getElementById('sidebar-nav-followed-tags')) {`
`94`	`93`	`const waitingForDataLoad = setInterval(() => {`
`95`	`94`	`const { user = null, userStatus } = document.body.dataset;`
`96`	`95`	`if (userStatus === 'logged-out') {`