From 12e3749aaa9652349c6f36c78535dbd52a814a5b Mon Sep 17 00:00:00 2001
From: Julia Miocene <jmiocene@gitlab.com>
Date: Wed, 17 Jul 2024 13:34:02 +0200
Subject: [PATCH] Migrate folder-o icon to folder

---
 .../environments/components/environment_folder.vue            | 2 +-
 .../jira_connect/subscriptions/components/group_item_name.vue | 2 +-
 app/helpers/icons_helper.rb                                   | 2 +-
 .../javascripts/approvals/components/hidden_groups_item.vue   | 2 +-
 .../frontend/approvals/components/hidden_groups_item_spec.js  | 2 +-
 spec/frontend/environments/environment_folder_spec.js         | 2 +-
 .../components/__snapshots__/group_item_name_spec.js.snap     | 2 +-
 spec/helpers/icons_helper_spec.rb                             | 4 ++--
 8 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/app/assets/javascripts/environments/components/environment_folder.vue b/app/assets/javascripts/environments/components/environment_folder.vue
index 2bf682cc23f8c1..2c6f62546f7663 100644
--- a/app/assets/javascripts/environments/components/environment_folder.vue
+++ b/app/assets/javascripts/environments/components/environment_folder.vue
@@ -60,7 +60,7 @@ export default {
     icons() {
       return this.visible
         ? { caret: 'chevron-lg-down', folder: 'folder-open' }
-        : { caret: 'chevron-lg-right', folder: 'folder-o' };
+        : { caret: 'chevron-lg-right', folder: 'folder' };
     },
     label() {
       return this.visible ? this.$options.i18n.collapse : this.$options.i18n.expand;
diff --git a/app/assets/javascripts/jira_connect/subscriptions/components/group_item_name.vue b/app/assets/javascripts/jira_connect/subscriptions/components/group_item_name.vue
index 2a52e5728f55b6..66dcbd4bec3ef4 100644
--- a/app/assets/javascripts/jira_connect/subscriptions/components/group_item_name.vue
+++ b/app/assets/javascripts/jira_connect/subscriptions/components/group_item_name.vue
@@ -19,7 +19,7 @@ export default {
 
 <template>
   <div class="gl-display-flex gl-align-items-center">
-    <gl-icon name="folder-o" class="gl-mr-3" />
+    <gl-icon name="folder" class="gl-mr-3" />
     <div class="gl-display-none gl-flex-shrink-0 gl-sm-display-flex gl-mr-3">
       <gl-avatar
         :size="32"
diff --git a/app/helpers/icons_helper.rb b/app/helpers/icons_helper.rb
index a3704dc6aa8c0d..00a0990c3705fc 100644
--- a/app/helpers/icons_helper.rb
+++ b/app/helpers/icons_helper.rb
@@ -126,7 +126,7 @@ def visibility_level_icon(level, options: {})
 
   def file_type_icon_class(type, mode, name)
     if type == 'folder'
-      'folder-o'
+      'folder'
     elsif type == 'archive'
       'archive'
     elsif mode == '120000'
diff --git a/ee/app/assets/javascripts/approvals/components/hidden_groups_item.vue b/ee/app/assets/javascripts/approvals/components/hidden_groups_item.vue
index b94796bbb727ea..20b54238a18ed9 100644
--- a/ee/app/assets/javascripts/approvals/components/hidden_groups_item.vue
+++ b/ee/app/assets/javascripts/approvals/components/hidden_groups_item.vue
@@ -14,7 +14,7 @@ export default {
 <template>
   <div class="gl-display-flex gl-align-items-center">
     <div class="square s24 gl-align-items-center gl-justify-content-center gl-mr-3 text-tertiary">
-      <gl-icon name="folder-o" :size="16" data-testid="folder-icon" />
+      <gl-icon name="folder" :size="16" data-testid="folder-icon" />
     </div>
     <span>{{ __('Private group(s)') }}</span>
     <gl-icon
diff --git a/ee/spec/frontend/approvals/components/hidden_groups_item_spec.js b/ee/spec/frontend/approvals/components/hidden_groups_item_spec.js
index 5f60cf14ee4706..9510b763624f51 100644
--- a/ee/spec/frontend/approvals/components/hidden_groups_item_spec.js
+++ b/ee/spec/frontend/approvals/components/hidden_groups_item_spec.js
@@ -38,7 +38,7 @@ describe('Approvals HiddenGroupsItem', () => {
     const folderIcon = findFolderIcon();
 
     expect(folderIcon.is(GlIcon)).toBe(true);
-    expect(folderIcon.props('name')).toBe('folder-o');
+    expect(folderIcon.props('name')).toBe('folder');
   });
 
   it('shows a help-icon with a tooltip', () => {
diff --git a/spec/frontend/environments/environment_folder_spec.js b/spec/frontend/environments/environment_folder_spec.js
index 61185a03c03adf..3e95a399e34ad4 100644
--- a/spec/frontend/environments/environment_folder_spec.js
+++ b/spec/frontend/environments/environment_folder_spec.js
@@ -89,7 +89,7 @@ describe('~/environments/components/environments_folder.vue', () => {
       it('is collapsed by default', () => {
         expect(collapse.props('visible')).toBe(false);
         const iconNames = icons.wrappers.map((i) => i.props('name')).slice(0, 2);
-        expect(iconNames).toEqual(['chevron-lg-right', 'folder-o']);
+        expect(iconNames).toEqual(['chevron-lg-right', 'folder']);
         expect(folderName.classes('gl-font-bold')).toBe(false);
       });
 
diff --git a/spec/frontend/jira_connect/subscriptions/components/__snapshots__/group_item_name_spec.js.snap b/spec/frontend/jira_connect/subscriptions/components/__snapshots__/group_item_name_spec.js.snap
index 59bc737a1bf61a..dd080e511a93ed 100644
--- a/spec/frontend/jira_connect/subscriptions/components/__snapshots__/group_item_name_spec.js.snap
+++ b/spec/frontend/jira_connect/subscriptions/components/__snapshots__/group_item_name_spec.js.snap
@@ -6,7 +6,7 @@ exports[`GroupItemName template matches the snapshot 1`] = `
 >
   <gl-icon-stub
     class="gl-mr-3"
-    name="folder-o"
+    name="folder"
     size="16"
     variant="current"
   />
diff --git a/spec/helpers/icons_helper_spec.rb b/spec/helpers/icons_helper_spec.rb
index f3732fa69abdbc..1a2a23d9be07b6 100644
--- a/spec/helpers/icons_helper_spec.rb
+++ b/spec/helpers/icons_helper_spec.rb
@@ -123,8 +123,8 @@
   end
 
   describe 'file_type_icon_class' do
-    it 'returns folder-o class' do
-      expect(file_type_icon_class('folder', 0, 'folder_name')).to eq 'folder-o'
+    it 'returns folder class' do
+      expect(file_type_icon_class('folder', 0, 'folder_name')).to eq 'folder'
     end
 
     it 'returns share class' do
-- 
GitLab