fix(changelog): simplify comparison UI and remove unused code in app.js

Author: vivekv1504

docs/changelog/assets/js/app.js

@@ -677,20 +677,10 @@ window.onhashchange = () => {
 
 populateVersions();
 
-let comparisonMode = false;
 /* ============================================
    UI HELPER FUNCTIONS
    ============================================ */
 
-/**
- * Show loading state for comparison
- */
-const showComparisonLoading = () => {
-  if (!comparisonResults) return;
-  comparisonResults.innerHTML = '<p style="text-align: center; padding: 20px;">Loading comparison...</p>';
-  comparisonResults.classList.remove('hide');
-};
-
 /**
  * Show error state for comparison
  * @param {Error} error - The error object
@@ -718,9 +708,6 @@ const showComparisonError = (error) => {
  * @param {string} versionB - Target version (optional)
  */
 const switchToComparisonMode = (versionA = null, versionB = null) => {
-  // Update mode
-  comparisonMode = true;
-
   // Update button states
   if (comparisonViewBtn && singleViewBtn) {
     comparisonViewBtn.classList.add('active', 'btn-primary');
@@ -972,18 +959,6 @@ const handleEnhancedComparisonURL = async () => {
 
   return {shouldCompare: false};
 };
-
-/**
- * Populate version dropdowns for comparison mode
- */
-const populateComparisonVersions = () => {
-  if (versionSelectDropdown && versionSelectDropdown.innerHTML) {
-    const options = versionSelectDropdown.innerHTML;
-    if (versionASelect) versionASelect.innerHTML = options;
-    if (versionBSelect) versionBSelect.innerHTML = options;
-  }
-};
-
 /**
  * Reset comparison form selections
  */
@@ -1067,7 +1042,6 @@ const updatePrereleaseLabels = () => {
  * Handle stable version changes - fetch changelogs and populate packages
  */
 const handleStableVersionChange = async () => {
-  console.log('🟢 handleStableVersionChange FIRED');
   const stableA = versionASelect.value;
   const stableB = versionBSelect.value;
 
@@ -1095,7 +1069,6 @@ const handleStableVersionChange = async () => {
  * Handle package selection - populate pre-release versions
  */
 const handlePackageChange = () => {
-  console.log('🟢 handlePackageChange FIRED');
   const selectedPackage = comparisonPackageSelect.value;
 
   if (versionAPrereleaseSelect) versionAPrereleaseSelect.value = '';
@@ -1130,8 +1103,6 @@ const handlePackageChange = () => {
  * Switch to single view mode
  */
 const switchToSingleViewMode = () => {
-  console.log('🔵 Switching to SINGLE VIEW mode');
-  comparisonMode = false;
 
   // Update button styles
   singleViewBtn.classList.add('active', 'btn-primary');
@@ -1145,25 +1116,6 @@ const switchToSingleViewMode = () => {
   clearComparisonURLParams();
 };
 
-/**
- * Switch to comparison view mode
- */
-const switchToComparisonViewMode = () => {
-  console.log('🔵 Switching to COMPARISON VIEW mode');
-  comparisonMode = true;
-
-  // Update button styles
-  comparisonViewBtn.classList.add('active', 'btn-primary');
-  comparisonViewBtn.classList.remove('btn-default');
-  singleViewBtn.classList.remove('active', 'btn-primary');
-  singleViewBtn.classList.add('btn-default');
-
-  // Toggle visibility (centralized view state)
-  updateUIVisibility('comparison');
-
-  populateComparisonVersions();
-};
-
 /**
  * Validate comparison form inputs
  */
@@ -1357,10 +1309,8 @@ const handleClearClick = () => {
  */
 const setupComparisonEventListeners = () => {
   if (comparisonListenersInitialized) {
-    console.log('🔴 Comparison listeners already initialized,skipping......');
     return;
   }
-  console.log('🟢 Setting up comparison event listeners first time......');
   comparisonListenersInitialized = true;
   // Mode toggle buttons
   if (singleViewBtn) singleViewBtn.addEventListener('click', switchToSingleViewMode);
@@ -1451,12 +1401,6 @@ const initializeComparisonMode = async () => {
     await loadEnhancedComparisonFromURL(enhancedParams);
     return;
   }
-
-  // Check for standard comparison URL
-  const urlParams = await handleComparisonURLParams();
-  if (urlParams.shouldCompare) {
-    await loadStandardComparisonFromURL(urlParams);
-  }
 };
 
 /**

docs/changelog/assets/js/comparison-view.js

@@ -294,8 +294,6 @@ const collectCommitsFromStable = (packageData, stableVersion, versionA, versionB
 const generatePackageComparisonData = (packageName, versionASpecific, versionBSpecific, changelogA, changelogB) => {
   const effectiveVersionA = getEffectiveVersion(changelogA, packageName, versionASpecific);
   const effectiveVersionB = getEffectiveVersion(changelogB, packageName, versionBSpecific);
-  console.log('effectiveVersionA', effectiveVersionA);
-  console.log('effectiveVersionB', effectiveVersionB);
   // Get package data from changelogs
   const pkgDataA = changelogA[packageName]?.[effectiveVersionA];
   const pkgDataB = changelogB[packageName]?.[effectiveVersionB];
@@ -334,18 +332,7 @@ const generatePackageComparisonData = (packageName, versionASpecific, versionBSp
 //Export All the functions
 export {
   comparisonState,
-  findLatestPackageVersion,
-  getEffectiveVersion,
-  getPackageVersion,
-  determinePackageStatus,
-  createPackageComparisonRow,
-  calculateComparisonStats,
-  buildPackagesList,
   generatePackageComparisonData,
   sortStableVersions,
-  isPreRelease,
-  isExactStable,
-  getPreReleaseNum,
-  getPreReleaseTag,
   collectCommitsFromStable,
 };

docs/changelog/index.html

@@ -158,11 +158,6 @@
                   <label for="comparison-package-select">Package for comparison:</label>
                   <select id="comparison-package-select" class="full-width">
                     <option value="">Select a package for comparison</option>
-=======
-                  <label for="comparison-package-select">Package (optional for full comparison):</label>
-                  <select id="comparison-package-select" class="full-width">
-                    <option value="">Select a package (optional)</option>
->>>>>>> de805712c1baa97a2d9aa67f0481d24cad266728
                   </select>
                 </div>
               </div>