Make sure minimap error indicators follow the minimap scrolling (#9624)

CHANGELOG.txt

@@ -40,6 +40,7 @@ Changelog
  * Fix: Ensure new tooltips & tooltip menus have visible borders and tip triangle in Windows high-contrast mode (Juliet Adeboye)
  * Fix: Ensure there is a visual difference of 'active/current link' vs normal links in Windows high-contrast mode (Mohammad Areeb)
  * Fix: Avoid issues where trailing whitespace could be accidentally removed in translations for new page & snippet headers (Florian Vogt)
+ * Fix: Make sure minimap error indicators follow the minimap scrolling (Thibaud Colas)
 
 
 4.1.1 (xx.xx.xxxx) - IN DEVELOPMENT
@@ -52,6 +53,7 @@ Changelog
  * Fix: Exclude tags from the reference index (Matt Westcott)
  * Fix: Fix errors in handling generic foreign keys when populating the reference index (Matt Westcott)
  * Fix: Prevent error in handling null ParentalKeys when populating the reference index (Matt Westcott)
+ * Fix: Make sure minimap error indicators follow the minimap scrolling (Thibaud Colas)
 
 
 4.1 LTS (01.11.2022)

client/src/components/Minimap/Minimap.scss

@@ -21,7 +21,6 @@ $minimap-z-index: calc(theme('zIndex.header') - 20);
   transform: translateX(calc(100% - $minimap-collapsed-width-mobile));
 
   @include media-breakpoint-up(sm) {
-    padding-inline-start: $minimap-overflow;
     transform: translateX(
       calc(100% - $minimap-collapsed-width - $minimap-overflow)
     );
@@ -97,6 +96,7 @@ $minimap-z-index: calc(theme('zIndex.header') - 20);
   min-height: 70px;
 
   :where(.w-minimap--expanded) & {
+    margin-inline-start: $minimap-overflow;
     border-inline-start: 1px solid theme('colors.grey.100');
   }
 }
@@ -104,6 +104,7 @@ $minimap-z-index: calc(theme('zIndex.header') - 20);
 .w-minimap__list {
   margin: 0;
   padding: 0;
+  padding-inline-start: $minimap-overflow;
   overflow-y: auto;
   overflow-x: hidden;
   list-style-type: none;
@@ -117,6 +118,7 @@ $minimap-z-index: calc(theme('zIndex.header') - 20);
   flex-grow: 1;
 
   :where(.w-minimap--expanded) & {
+    margin-inline-start: $minimap-overflow;
     border-inline-start: 1px solid theme('colors.grey.100');
   }
 }

client/src/components/Minimap/MinimapItem.scss

@@ -76,7 +76,6 @@
 .w-minimap-item__errors {
   $border-width: 1px;
   $badge-size: theme('spacing.4');
-  position: absolute;
   width: $badge-size;
   height: $badge-size;
   flex-shrink: 0;
@@ -93,5 +92,6 @@
     margin-inline-start: calc(-1 * (theme('spacing.8') + $badge-size / 2));
     // Improve the alignment of "1" with the item’s border.
     padding-inline-end: theme('spacing.px');
+    margin-inline-end: calc($badge-size - theme('spacing.px'));
   }
 }

docs/releases/4.1.1.md

@@ -22,3 +22,4 @@ depth: 1
  * Exclude tags from the reference index (Matt Westcott)
  * Fix errors in handling generic foreign keys when populating the reference index (Matt Westcott)
  * Prevent error in handling null ParentalKeys when populating the reference index (Matt Westcott)
+ * Make sure minimap error indicators follow the minimap scrolling (Thibaud Colas)

docs/releases/4.2.md

@@ -51,6 +51,7 @@ depth: 1
  * Ensure new tooltips & tooltip menus have visible borders and tip triangle in Windows high-contrast mode (Juliet Adeboye)
  * Ensure there is a visual difference of 'active/current link' vs normal links in Windows high-contrast mode (Mohammad Areeb)
  * Avoid issues where trailing whitespace could be accidentally removed in translations for new page & snippet headers (Florian Vogt)
+ * Make sure minimap error indicators follow the minimap scrolling (Thibaud Colas)
 
 ## Upgrade considerations