Merge pull request #1153 from skyash-dev/docs/multiple-small-fixes-2.0

fix: multiple small documentation issues across tutorials and reference

Author: davepagurek

src/components/PageHeader/RootPage.astro

@@ -12,7 +12,7 @@ const borderStyle = isFilterRoute ? "" : "border-b border-sidebar-type-color";
 ---
 
 <div
-  class=`content-grid-simple min-h-[350px] h-[50vh] pt-[11.25rem] lg:pt-4xl bg-accent-color text-accent-type-color pb-md px-5 md:px-lg ${borderStyle}`
+  class=`content-grid-simple min-h-[350px] pt-[11.25rem] lg:pt-4xl bg-accent-color text-accent-type-color pb-md px-5 md:px-lg ${borderStyle}`
 >
   <h1 class="whitespace-pre-line col-span-full mb-5">{title}</h1>
   <p class="text-h3 !mt-0 mb-3xl col-span-2">{subtitle}</p>

src/content/tutorials/en/conditionals-and-interactivity.mdx

@@ -23,7 +23,7 @@ import { Columns, Column } from "../../../components/Columns";
 import AnnotatedLine from "../../../components/AnnotatedLine/index.astro";
 import Video from "../../../components/Video/index.astro"
 
-In this tutorial, you will learn new ways to add user interaction to your sketches, and control the order in which code runs. 
+In this tutorial, you will learn new ways to add user interaction to your sketches, and control the order in which code runs.
 
 You will learn these basic programming concepts by creating an [interactive sun](https://editor.p5js.org/gbenedis@gmail.com/sketches/nNVmHVf5m) sketch and a [sunrise animation](https://editor.p5js.org/gbenedis@gmail.com/sketches/9lz2aqfTO):
 
@@ -524,7 +524,7 @@ function draw() {
 
   fill("green");
 
-  rect(0, horizon, 400, 400);
+  rect(0, horizon, 400, 200);
 
 }
 ```
@@ -764,7 +764,7 @@ function draw() {
     sunHeight -= 2;
   }
 
-  // increase color variables by 4 or 1 during sunrise 
+  // increase color variables by 4 or 1 during sunrise
   if (sunHeight < 480) {
     redVal += 4;
     greenVal += 1;

src/content/tutorials/en/repeating-with-loops.mdx

@@ -199,7 +199,7 @@ Above `setup()`:
 In `draw()`:
 
 - After the code that draws the finish line, declare a new local variable `x` to position all the body segments: `let x = circX;`
-- Add a *for loop* using: `for (let i = 0; i < length; i += 1) { }`
+- Add a *for loop* using: `for (let i = 0; i < segments; i += 1) { }`
   - A *for loop* will repeat the code we write inside the curly brackets multiple times. 
   - Move the lines of code that draw the `circle()` into the curly brackets of the *for loop*.
 - After the for loop, add: `circX += spacing` 
@@ -936,7 +936,7 @@ function moveCaterpillars() {
   for (let i = 0; i < numCaterpillars; i += 1) {
     //Give each caterpillar a
     //random speed.
-    move = random(5, 30);
+    let move = random(5, 30);
     caterpillarEnds[i] += move;
   }
 }
@@ -1001,7 +1001,7 @@ function moveCaterpillars() {
   for (let i = 0; i < numCaterpillars; i += 1) {
     //Give each caterpillar a
     //random speed.
-    move = random(5, 30);
+    let move = random(5, 30);
 
     // Update caterpillars' x-coordinates
     caterpillarEnds[i] += move;
@@ -1356,7 +1356,7 @@ function drawCaterpillar(x, y, segments) {
 function drawCaterpillars() {
   let padding = height / numCaterpillars;
   for (let i = 0; i < numCaterpillars; i += 1) {
-    // Update each caterpillar x-coordinate   
+    // Update each caterpillar x-coordinate  
     let y = (i + 0.5) * padding;
 
     // Update length of caterpillar.   
@@ -1379,7 +1379,7 @@ function moveCaterpillars() {
     // Give each caterpillar a random
     // speed once the race has started.
     let move = round(random(5, 30));
-   
+  
     // Update caterpillars' x-coordinates
     caterpillarEnds[i] += move;
   }

src/pages/_utils-node.ts

@@ -45,13 +45,13 @@ export const removeNestedReferencePaths = (route: string): string =>
   route.replace(/types\//, "")
 
 export const rewriteRelativeLink = (url: string): string => {
-  let updatedUrl: string;
+  let updatedUrl: string = url;
 
-  if (/^((https?:\/)?)\//.exec(url) || url.startsWith('mailto:')) {
-    // Leave absolute paths alone
-    updatedUrl = url;
-  } else if (url.startsWith('#')) {
-    // Leave links to headings alone
+  if (/^(https?:|mailto:|\/\/)/.exec(url)) {
+    // Skip rewriting for external URLs (http(s), mailto)
+    return url;
+  } else if (url.startsWith('#')||url.startsWith('/')) {
+    // Skip rewriting for heading links and root-relative internal paths
     updatedUrl = url;
   } else {
     // Convert relative paths to '../' (because pages that started as files in the same directory
@@ -81,4 +81,4 @@ export const rewriteRelativeLink = (url: string): string => {
   }
 
   return updatedUrl;
-};
+};