YAGNI: Remove support for {% all %}

Simran-B · Simran-B · commit 3f8f75e116de · 2020-12-11T01:05:23.000+01:00
diff --git a/contributing/content-markup-reference.md b/contributing/content-markup-reference.md
@@ -38,10 +38,10 @@ For information on when to use callout tags, see the [style guide](content-style
 
 To render syntax highlighting in command line instructions, we use triple backticks followed by the term `shell`.
 
-### Usage  
-  
-    ```shell  
-    git init <em>YOUR_REPO</em>  
+### Usage
+
+    ```shell
+    git init <em>YOUR_REPO</em>
     ```
 
 This syntax highlighting renders light text on a dark background, and should be reserved for command line instructions.
@@ -93,13 +93,6 @@ These instructions are pertinent to Windows users.
 
 {% endlinux %}
 ```
-```
-{% all %}
-
-Though rare, these instructions are pertinent to any other operating system.
-
-{% endall %}
-```
 
 ## Reusable and variable strings of text
 
diff --git a/includes/article.html b/includes/article.html
@@ -35,7 +35,6 @@ <h1 class="border-bottom-0">{{ page.title }}</h1>
             <a href="#" class="UnderlineNav-item platform-switcher" data-platform="mac">Mac</a>
             <a href="#" class="UnderlineNav-item platform-switcher" data-platform="windows">Windows</a>
             <a href="#" class="UnderlineNav-item platform-switcher" data-platform="linux">Linux</a>
-            <a href="#" class="UnderlineNav-item platform-switcher" data-platform="all">All</a>
           </div>
         </nav>
       {% endif %}
diff --git a/lib/frontmatter.js b/lib/frontmatter.js
@@ -104,7 +104,7 @@ const schema = {
     // Platform-specific content preference
     defaultPlatform: {
       type: 'string',
-      enum: ['mac', 'windows', 'linux'] // TODO: add 'all'?
+      enum: ['mac', 'windows', 'linux']
     }
   }
 }
diff --git a/lib/page.js b/lib/page.js
@@ -189,7 +189,7 @@ class Page {
     const $ = cheerio.load(html)
 
     // set a flag so layout knows whether to render a mac/windows/linux switcher element
-    this.includesPlatformSpecificContent = $('[class^="platform-"], .mac, .windows, .linux, .all').length > 0
+    this.includesPlatformSpecificContent = $('[class^="platform-"], .mac, .windows, .linux').length > 0
 
     // rewrite asset paths to s3 if it's a dotcom article on any GHE version
     // or if it's an enterprise article on any GHE version EXCEPT latest version

Original file line number	Diff line number	Diff line change
`@@ -104,7 +104,7 @@ const schema = {`
`104`	`104`	`// Platform-specific content preference`
`105`	`105`	`defaultPlatform: {`
`106`	`106`	`type: 'string',`
`107`		`- enum: ['mac', 'windows', 'linux'] // TODO: add 'all'?`
	`107`	`+ enum: ['mac', 'windows', 'linux']`
`108`	`108`	`}`
`109`	`109`	`}`
`110`	`110`	`}`