From 1bd2bca929365f393cd54154f393641ed575f084 Mon Sep 17 00:00:00 2001
From: sopa301 <96387349+sopa301@users.noreply.github.com>
Date: Tue, 2 Apr 2024 20:13:17 +0800
Subject: [PATCH 01/11] Fix inconsistent line number colours

---
 frontend/src/components/c-segment.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/frontend/src/components/c-segment.vue b/frontend/src/components/c-segment.vue
index ef390a541c..73c454cb52 100644
--- a/frontend/src/components/c-segment.vue
+++ b/frontend/src/components/c-segment.vue
@@ -69,7 +69,7 @@ export default defineComponent({
 });
 </script>
 
-<style lang="scss" scoped>
+<style lang="scss">
 @import '../styles/hightlight-js-style.css';
 @import '../styles/_colors.scss';
 

From e236ad2ce883587912ea45f2d9d9e5adfd14d979 Mon Sep 17 00:00:00 2001
From: sopa301 <96387349+sopa301@users.noreply.github.com>
Date: Tue, 2 Apr 2024 20:27:23 +0800
Subject: [PATCH 02/11] Add tests

---
 .../codeView/codeView_codeHighlighting.cy.js  | 21 +++++++++++++++++++
 1 file changed, 21 insertions(+)
 create mode 100644 frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js

diff --git a/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js b/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
new file mode 100644
index 0000000000..9fdda44ac6
--- /dev/null
+++ b/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
@@ -0,0 +1,21 @@
+describe('code highlighting works properly', () => {
+  it('line numbers should all have the same colour', () => {
+    cy.get('.icon-button.fa-code')
+      .should('exist')
+      .first()
+      .click();
+
+    cy.get('#tab-authorship .files', { timeout: 90000 })
+      .should('be.visible');
+
+    cy.get('.line-number')
+      .should('have.css', 'color', 'rgb(158, 158, 158)') // #9E9E9E
+      .then((color) => {
+        const firstColor = color;
+        cy.get('.line-number')
+          .each(($el) => {
+            expect($el.css('color')).to.equal(firstColor);
+          });
+      });
+  })
+});
\ No newline at end of file

From a796afd15f8721ebc33e8185d7d46bede0a99877 Mon Sep 17 00:00:00 2001
From: sopa301 <96387349+sopa301@users.noreply.github.com>
Date: Tue, 2 Apr 2024 20:31:32 +0800
Subject: [PATCH 03/11] Fix style

---
 frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js b/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
index 9fdda44ac6..91656c3d78 100644
--- a/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
+++ b/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
@@ -18,4 +18,4 @@ describe('code highlighting works properly', () => {
           });
       });
   })
-});
\ No newline at end of file
+});

From fe875a69f5876ec9d4c61e0133ca229653fe9c87 Mon Sep 17 00:00:00 2001
From: sopa301 <96387349+sopa301@users.noreply.github.com>
Date: Tue, 2 Apr 2024 20:33:53 +0800
Subject: [PATCH 04/11] Fix style

---
 frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js b/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
index 91656c3d78..ffe60f327c 100644
--- a/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
+++ b/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
@@ -17,5 +17,5 @@ describe('code highlighting works properly', () => {
             expect($el.css('color')).to.equal(firstColor);
           });
       });
-  })
+  });
 });

From 3e625540c48692f3601719b06ec00511380a60d1 Mon Sep 17 00:00:00 2001
From: sopa301 <96387349+sopa301@users.noreply.github.com>
Date: Tue, 2 Apr 2024 20:47:07 +0800
Subject: [PATCH 05/11] Fix test

---
 .../tests/codeView/codeView_codeHighlighting.cy.js       | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js b/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
index ffe60f327c..c95a6cdcf3 100644
--- a/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
+++ b/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
@@ -9,13 +9,8 @@ describe('code highlighting works properly', () => {
       .should('be.visible');
 
     cy.get('.line-number')
-      .should('have.css', 'color', 'rgb(158, 158, 158)') // #9E9E9E
-      .then((color) => {
-        const firstColor = color;
-        cy.get('.line-number')
-          .each(($el) => {
-            expect($el.css('color')).to.equal(firstColor);
-          });
+      .each((el) => {
+        expect(el.css('color')).to.equal('rgb(158, 158, 158)'); // #9E9E9E
       });
   });
 });

From ea100cef9f6ebf9fc75d14088f909126002319d3 Mon Sep 17 00:00:00 2001
From: sopa301 <96387349+sopa301@users.noreply.github.com>
Date: Sat, 6 Apr 2024 21:47:51 +0800
Subject: [PATCH 06/11] Modify test

---
 .../tests/codeView/codeView_codeHighlighting.cy.js  | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js b/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
index c95a6cdcf3..6b44224d33 100644
--- a/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
+++ b/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
@@ -9,8 +9,15 @@ describe('code highlighting works properly', () => {
       .should('be.visible');
 
     cy.get('.line-number')
-      .each((el) => {
-        expect(el.css('color')).to.equal('rgb(158, 158, 158)'); // #9E9E9E
-      });
+      .first()
+      .expect('to.have.css', 'color')
+      .then((firstColor) => {
+        cy.get('.line-number')
+          .each((el) => {
+            expect(el.css('color')).to.equal(firstColor);
+          }
+        )
+      }
+    );
   });
 });

From 3458ac286f97349613f73775f1a82c0240cfd1ca Mon Sep 17 00:00:00 2001
From: sopa301 <96387349+sopa301@users.noreply.github.com>
Date: Sat, 6 Apr 2024 21:52:56 +0800
Subject: [PATCH 07/11] Fix style

---
 frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js b/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
index 6b44224d33..ef1b08d1ac 100644
--- a/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
+++ b/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
@@ -15,7 +15,7 @@ describe('code highlighting works properly', () => {
         cy.get('.line-number')
           .each((el) => {
             expect(el.css('color')).to.equal(firstColor);
-          }
+          },
         )
       }
     );

From c8fcf8e4215697b5a272619751a5eb4641da350e Mon Sep 17 00:00:00 2001
From: sopa301 <96387349+sopa301@users.noreply.github.com>
Date: Sat, 6 Apr 2024 21:54:30 +0800
Subject: [PATCH 08/11] Fix test

---
 frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js b/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
index ef1b08d1ac..93f1bc31e3 100644
--- a/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
+++ b/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
@@ -10,7 +10,7 @@ describe('code highlighting works properly', () => {
 
     cy.get('.line-number')
       .first()
-      .expect('to.have.css', 'color')
+      .should('have.css', 'color')
       .then((firstColor) => {
         cy.get('.line-number')
           .each((el) => {

From f8bee12fc5e3734d02a2dbc849e87f9d5a87ca36 Mon Sep 17 00:00:00 2001
From: sopa301 <96387349+sopa301@users.noreply.github.com>
Date: Sat, 6 Apr 2024 21:59:58 +0800
Subject: [PATCH 09/11] Fix style

---
 .../tests/codeView/codeView_codeHighlighting.cy.js        | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js b/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
index 93f1bc31e3..a82a327c9b 100644
--- a/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
+++ b/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
@@ -13,11 +13,7 @@ describe('code highlighting works properly', () => {
       .should('have.css', 'color')
       .then((firstColor) => {
         cy.get('.line-number')
-          .each((el) => {
-            expect(el.css('color')).to.equal(firstColor);
-          },
-        )
-      }
-    );
+          .each((el) => expect(el.css('color')).to.equal(firstColor));
+      });
   });
 });

From edc81673ae264474f2d40a2644212a665efbbfc2 Mon Sep 17 00:00:00 2001
From: sopa301 <96387349+sopa301@users.noreply.github.com>
Date: Sat, 13 Apr 2024 19:00:16 +0800
Subject: [PATCH 10/11] Fix test

---
 .../tests/codeView/codeView_codeHighlighting.cy.js     | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js b/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
index a82a327c9b..69add80ce5 100644
--- a/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
+++ b/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
@@ -8,12 +8,18 @@ describe('code highlighting works properly', () => {
     cy.get('#tab-authorship .files', { timeout: 90000 })
       .should('be.visible');
 
-    cy.get('.line-number')
+    cy.get('.line-number') // this is just a wrapper
+      .first()
+      .children()
+      // the actual line number element
       .first()
       .should('have.css', 'color')
       .then((firstColor) => {
         cy.get('.line-number')
-          .each((el) => expect(el.css('color')).to.equal(firstColor));
+          .each((el) => cy.wrap(el)
+            .children()
+            .first()
+            .should('have.css', 'color', firstColor));
       });
   });
 });

From 0bbdefd3a4229c7afbfbff69fe3a1f3f2aac02f1 Mon Sep 17 00:00:00 2001
From: sopa301 <96387349+sopa301@users.noreply.github.com>
Date: Sat, 13 Apr 2024 19:06:36 +0800
Subject: [PATCH 11/11] Add comment

---
 .../cypress/tests/codeView/codeView_codeHighlighting.cy.js     | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js b/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
index 69add80ce5..76066cbdc7 100644
--- a/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
+++ b/frontend/cypress/tests/codeView/codeView_codeHighlighting.cy.js
@@ -11,7 +11,8 @@ describe('code highlighting works properly', () => {
     cy.get('.line-number') // this is just a wrapper
       .first()
       .children()
-      // the actual line number element
+      // The actual line number element. We can't name the class
+      // because it's generated by the code highlighter.
       .first()
       .should('have.css', 'color')
       .then((firstColor) => {