Skip to content

Replace `<i>` in vue components with `<icon>`

What does this MR do?

Replaces <i> to <icon> in vue components. This includes to changes to following vue templates (and one haml template to avoid inconsistency between dismisable banners):

  • issue_card_inner.vue
  • boards/components/issue_card_inner.vue
  • boards/components/modal/list.vue
  • boards/components/modal/lists_dropdown.vue
  • boards/components/project_select.vue
  • cycle_analytics/components/banner.vue
  • diffs/components/compare_versions_dropdown.vue
  • environments/components/environment_actions.vue
  • environments/components/environment_item.vue
  • frequent_items/components/frequent_items_search_input.vue
  • vue_shared/components/notes/system_note.vue
  • app/views/explore/groups/index.html.haml

Are there points in the code the reviewer needs to double check?

See discussions below.

Screenshots (if relevant)

Before After
board-issues-dropdown-before board-issues-dropdown-after
cycle-analytics-before cycle-analytics-after
confidential-issue-before confidential-issue-after
env-actions-before env-actions-after
env-folder-before env-folder-after
project-select-before project-select-after

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

This is related to #44646 (moved). Some sections were left behind (like Web IDE icons), to avoid making this merge request too large. Second iteration should replace more <i>s.

Edited by George Tsiolis

Merge request reports