Merge pull request #403 from snehaljha-sf/file_diff_change

feat: file diff td with new UX changes

Author: snehaljha-sf

src/styles/reportGenerator.css

@@ -306,10 +306,11 @@ html {
 .expandModalButton {
   position: absolute;
   top: 5px;
-  right: 5px;
+  right: 1.5rem;
   width: 30px;
   height: 30px;
   opacity: 0.7;
+  display: none;
 }
 .report-wrapper {
   display: flex;
@@ -574,3 +575,11 @@ td {
 .normal-td-padding {
   padding-left: 0.5rem !important;
 }
+
+.diff-cell {
+  min-width: 30rem;
+}
+
+.diff-cell:hover .expandModalButton {
+  display: block;
+}

src/utils/resultsbuilder/ApexAssessmentReporter.ts

@@ -108,7 +108,8 @@ export class ApexAssessmentReporter {
           1,
           false,
           undefined,
-          FileDiffUtil.getDiffHTML(apexAssessmentInfo.diff, apexAssessmentInfo.name)
+          FileDiffUtil.getDiffHTML(apexAssessmentInfo.diff, apexAssessmentInfo.name),
+          'diff-cell'
         ),
         createRowDataParam(
           'comments',

src/utils/resultsbuilder/ExperienceSiteAssessmentReporter.ts

@@ -119,7 +119,8 @@ export class ExperienceSiteAssessmentReporter {
           1,
           false,
           undefined,
-          FileDiffUtil.getDiffHTML(experienceSiteAssessmentInfo.diff, experienceSiteAssessmentInfo.name)
+          FileDiffUtil.getDiffHTML(experienceSiteAssessmentInfo.diff, experienceSiteAssessmentInfo.name),
+          'diff-cell'
         ),
         createRowDataParam('summary', '', false, 1, 1, false, undefined, experienceSiteAssessmentInfo.warnings),
       ],

src/utils/resultsbuilder/FlexipageAssessmentReporter.ts

@@ -142,7 +142,8 @@ export class FlexipageAssessmentReporter {
           1,
           false,
           undefined,
-          FileDiffUtil.getDiffHTML(flexipageAssessmentInfo.diff, flexipageAssessmentInfo.name)
+          FileDiffUtil.getDiffHTML(flexipageAssessmentInfo.diff, flexipageAssessmentInfo.name),
+          'diff-cell'
         ),
         createRowDataParam(
           'errors',

src/utils/resultsbuilder/LWCAssessmentReporter.ts

@@ -111,7 +111,8 @@ export class LWCAssessmentReporter {
               1,
               false,
               undefined,
-              FileDiffUtil.getDiffHTML(fileChangeInfo.diff, fileChangeInfo.name)
+              FileDiffUtil.getDiffHTML(fileChangeInfo.diff, fileChangeInfo.name),
+              'diff-cell'
             ),
             ...(fileIndex === 0
               ? [

src/utils/resultsbuilder/index.ts

@@ -398,7 +398,8 @@ export class ResultsBuilder {
             1,
             false,
             undefined,
-            FileDiffUtil.getDiffHTML(item.diff, this.rowId.toString())
+            FileDiffUtil.getDiffHTML(item.diff, this.rowId.toString()),
+            'diff-cell'
           ),
           createRowDataParam(
             'errors',
@@ -493,7 +494,17 @@ export class ResultsBuilder {
             undefined,
             item.status === 'Successfully migrated' ? 'text-success' : 'text-error'
           ),
-          createRowDataParam('diff', '', false, 1, 1, false, undefined, FileDiffUtil.getDiffHTML(item.diff, item.name)),
+          createRowDataParam(
+            'diff',
+            '',
+            false,
+            1,
+            1,
+            false,
+            undefined,
+            FileDiffUtil.getDiffHTML(item.diff, item.name),
+            'diff-cell'
+          ),
           createRowDataParam('error', 'error', false, 1, 1, false, undefined, item.errors),
         ],
       })),
@@ -583,7 +594,8 @@ export class ResultsBuilder {
             1,
             false,
             undefined,
-            FileDiffUtil.getDiffHTML(item.diff, item.name)
+            FileDiffUtil.getDiffHTML(item.diff, item.name),
+            'diff-cell'
           ),
           createRowDataParam('infos', item.infos ? item.infos.join(', ') : '', false, 1, 1, false, undefined, [
             ...item.infos,
@@ -719,7 +731,8 @@ export class ResultsBuilder {
               1,
               false,
               undefined,
-              FileDiffUtil.getDiffHTML(fileChangeInfo.diff, fileChangeInfo.name)
+              FileDiffUtil.getDiffHTML(fileChangeInfo.diff, fileChangeInfo.name),
+              'diff-cell'
             ),
             ...(showCommon
               ? [