Delete Primer::Alpha::ImageCrop (#3158)

jonrohan · web-flow · commit 683abd2bc83f · 2024-10-18T16:32:42.000-07:00
diff --git a/.changeset/afraid-teachers-battle.md b/.changeset/afraid-teachers-battle.md
@@ -0,0 +1,5 @@
+---
+"@primer/view-components": minor
+---
+
+Delete Primer::Alpha::ImageCrop
diff --git a/app/components/primer/alpha/image_crop.html.erb b/app/components/primer/alpha/image_crop.html.erb
diff --git a/app/components/primer/alpha/image_crop.rb b/app/components/primer/alpha/image_crop.rb
diff --git a/app/components/primer/alpha/image_crop.ts b/app/components/primer/alpha/image_crop.ts
diff --git a/app/components/primer/primer.ts b/app/components/primer/primer.ts
@@ -9,7 +9,6 @@ import './focus_group'
 import './scrollable_region'
 import './aria_live'
 import './shared_events'
-import './alpha/image_crop'
 import './alpha/modal_dialog'
 import './beta/nav_list'
 import './beta/nav_list_group_element'
diff --git a/lib/primer/yard/component_manifest.rb b/lib/primer/yard/component_manifest.rb
@@ -14,7 +14,6 @@ class ComponentManifest
         Primer::Alpha::HellipButton => {},
         Primer::Alpha::Image => {},
         Primer::Alpha::OcticonSymbols => {},
-        Primer::Alpha::ImageCrop => { js: true },
         Primer::IconButton => { js: true },
         Primer::Beta::AutoComplete => { js: true },
         Primer::Beta::AutoComplete::Item => {},
diff --git a/previews/primer/alpha/image_crop_preview.rb b/previews/primer/alpha/image_crop_preview.rb
diff --git a/static/arguments.json b/static/arguments.json
@@ -1351,34 +1351,6 @@
       }
     ]
   },
-  {
-    "component": "ImageCrop",
-    "status": "alpha",
-    "a11y_reviewed": false,
-    "short_name": "ImageCrop",
-    "source": "https://github.com/primer/view_components/tree/main/app/components/primer/alpha/image_crop.rb",
-    "lookbook": "https://primer.style/view-components/lookbook/inspect/primer/alpha/image_crop/default/",
-    "parameters": [
-      {
-        "name": "src",
-        "type": "String",
-        "default": "N/A",
-        "description": "The path of the image to crop."
-      },
-      {
-        "name": "rounded",
-        "type": "Boolean",
-        "default": "`true`",
-        "description": "If the crop mask should be a circle. Defaults to true."
-      },
-      {
-        "name": "system_arguments",
-        "type": "Hash",
-        "default": "N/A",
-        "description": "[System arguments](/system-arguments)"
-      }
-    ]
-  },
   {
     "component": "Layout",
     "status": "alpha",
diff --git a/static/audited_at.json b/static/audited_at.json
@@ -30,7 +30,6 @@
   "Primer::Alpha::HellipButton": "",
   "Primer::Alpha::HiddenTextExpander": "",
   "Primer::Alpha::Image": "",
-  "Primer::Alpha::ImageCrop": "",
   "Primer::Alpha::Layout": "",
   "Primer::Alpha::Layout::Main": "",
   "Primer::Alpha::Layout::Sidebar": "",
diff --git a/static/constants.json b/static/constants.json
@@ -356,8 +356,6 @@
   },
   "Primer::Alpha::Image": {
   },
-  "Primer::Alpha::ImageCrop": {
-  },
   "Primer::Alpha::Layout": {
     "FIRST_IN_SOURCE_DEFAULT": "sidebar",
     "FIRST_IN_SOURCE_OPTIONS": [
diff --git a/static/info_arch.json b/static/info_arch.json
@@ -4620,101 +4620,6 @@
 
     ]
   },
-  {
-    "fully_qualified_name": "Primer::Alpha::ImageCrop",
-    "description": "A client-side mechanism to crop images.",
-    "accessibility_docs": null,
-    "is_form_component": false,
-    "is_published": true,
-    "requires_js": true,
-    "component": "ImageCrop",
-    "status": "alpha",
-    "a11y_reviewed": false,
-    "short_name": "ImageCrop",
-    "source": "https://github.com/primer/view_components/tree/main/app/components/primer/alpha/image_crop.rb",
-    "lookbook": "https://primer.style/view-components/lookbook/inspect/primer/alpha/image_crop/default/",
-    "parameters": [
-      {
-        "name": "src",
-        "type": "String",
-        "default": "N/A",
-        "description": "The path of the image to crop."
-      },
-      {
-        "name": "rounded",
-        "type": "Boolean",
-        "default": "`true`",
-        "description": "If the crop mask should be a circle. Defaults to true."
-      },
-      {
-        "name": "system_arguments",
-        "type": "Hash",
-        "default": "N/A",
-        "description": "{{link_to_system_arguments_docs}}"
-      }
-    ],
-    "slots": [
-      {
-        "name": "loading",
-        "description": "A loading indicator that is shown while the image is loading.",
-        "parameters": [
-          {
-            "name": "system_arguments",
-            "type": "Hash",
-            "default": "N/A",
-            "description": "{{link_to_system_arguments_docs}}"
-          }
-        ]
-      }
-    ],
-    "methods": [
-
-    ],
-    "previews": [
-      {
-        "preview_path": "primer/alpha/image_crop/playground",
-        "name": "playground",
-        "snapshot": "false",
-        "skip_rules": {
-          "wont_fix": [
-            "region"
-          ],
-          "will_fix": [
-            "color-contrast"
-          ]
-        }
-      },
-      {
-        "preview_path": "primer/alpha/image_crop/default",
-        "name": "default",
-        "snapshot": "false",
-        "skip_rules": {
-          "wont_fix": [
-            "region"
-          ],
-          "will_fix": [
-            "color-contrast"
-          ]
-        }
-      },
-      {
-        "preview_path": "primer/alpha/image_crop/loading",
-        "name": "loading",
-        "snapshot": "false",
-        "skip_rules": {
-          "wont_fix": [
-            "region"
-          ],
-          "will_fix": [
-            "color-contrast"
-          ]
-        }
-      }
-    ],
-    "subcomponents": [
-
-    ]
-  },
   {
     "fully_qualified_name": "Primer::Alpha::Layout",
     "description": "`Layout` provides foundational patterns for responsive pages.\n`Layout` can be used for simple two-column pages, or it can be nested to provide flexible 3-column experiences.\n On smaller screens, `Layout` uses vertically stacked rows to display content.\n\n`Layout` flows as both column, when there's enough horizontal space to render both `Main` and `Sidebar`side-by-side (on a desktop of tablet device, per instance);\nor it flows as a row, when `Main` and `Sidebar` are stacked vertically (e.g. on a mobile device).\n`Layout` should always work in any screen size.",
diff --git a/static/previews.json b/static/previews.json
@@ -3978,53 +3978,6 @@
       }
     ]
   },
-  {
-    "name": "image_crop",
-    "component": "ImageCrop",
-    "status": "alpha",
-    "lookup_path": "primer/alpha/image_crop",
-    "examples": [
-      {
-        "preview_path": "primer/alpha/image_crop/playground",
-        "name": "playground",
-        "snapshot": "false",
-        "skip_rules": {
-          "wont_fix": [
-            "region"
-          ],
-          "will_fix": [
-            "color-contrast"
-          ]
-        }
-      },
-      {
-        "preview_path": "primer/alpha/image_crop/default",
-        "name": "default",
-        "snapshot": "false",
-        "skip_rules": {
-          "wont_fix": [
-            "region"
-          ],
-          "will_fix": [
-            "color-contrast"
-          ]
-        }
-      },
-      {
-        "preview_path": "primer/alpha/image_crop/loading",
-        "name": "loading",
-        "snapshot": "false",
-        "skip_rules": {
-          "wont_fix": [
-            "region"
-          ],
-          "will_fix": [
-            "color-contrast"
-          ]
-        }
-      }
-    ]
-  },
   {
     "name": "label",
     "component": "Label",
diff --git a/static/statuses.json b/static/statuses.json
@@ -30,7 +30,6 @@
   "Primer::Alpha::HellipButton": "alpha",
   "Primer::Alpha::HiddenTextExpander": "alpha",
   "Primer::Alpha::Image": "alpha",
-  "Primer::Alpha::ImageCrop": "alpha",
   "Primer::Alpha::Layout": "alpha",
   "Primer::Alpha::Layout::Main": "alpha",
   "Primer::Alpha::Layout::Sidebar": "alpha",
diff --git a/test/components/alpha/image_crop_test.rb b/test/components/alpha/image_crop_test.rb
diff --git a/test/components/component_test.rb b/test/components/component_test.rb
@@ -33,7 +33,6 @@ class PrimerComponentTest < Minitest::Test
     [Primer::Alpha::TabNav, { label: "label" }],
     [Primer::Alpha::UnderlinePanels, { label: "Panel label" }],
     [Primer::Alpha::Image, { src: "https://github.com/github.png", alt: "alt" }],
-    [Primer::Alpha::ImageCrop, { src: "Foo" }],
     [Primer::IconButton, { icon: :star, "aria-label": "Label" }],
     [Primer::Alpha::ActionList, { aria: { label: "Action List" } }, lambda do |component|
       component.with_item(label: "Foo")

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +---
 +"@primer/view-components": minor
 +---
++
 +Delete Primer::Alpha::ImageCrop