From 818dcea861e1c46945bfb1a780cae0f965cbf2ce Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tu=C4=9F=C3=A7e=20K=C3=BC=C3=A7=C3=BCko=C4=9Flu?=
 <tugcekucukoglu.860@gmail.com>
Date: Tue, 28 Jun 2022 17:21:35 +0300
Subject: [PATCH] Fixed #2695 - Menu, Menubar, ContextMenu, TieredMenu: Extra
 spacing when no icon

---
 src/components/contextmenu/ContextMenuSub.vue | 4 ++--
 src/components/menu/Menuitem.vue              | 4 ++--
 src/components/menubar/MenubarSub.vue         | 4 ++--
 src/components/tieredmenu/TieredMenuSub.vue   | 4 ++--
 4 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/components/contextmenu/ContextMenuSub.vue b/src/components/contextmenu/ContextMenuSub.vue
index ea3fb303d..b8baffe8d 100755
--- a/src/components/contextmenu/ContextMenuSub.vue
+++ b/src/components/contextmenu/ContextMenuSub.vue
@@ -7,13 +7,13 @@
                     <template v-if="!template">
                         <router-link v-if="item.to && !disabled(item)" :to="item.to" custom v-slot="{navigate, href, isActive, isExactActive}">
                             <a :href="href" @click="onItemClick($event, item, navigate)" :class="linkClass(item, {isActive, isExactActive})" v-ripple role="menuitem">
-                                <span :class="['p-menuitem-icon', item.icon]"></span>
+                                <span :class="['p-menuitem-icon', item.icon]" v-if="item.icon"></span>
                                 <span class="p-menuitem-text">{{label(item)}}</span>
                             </a>
                         </router-link>
                         <a v-else :href="item.url" :class="linkClass(item)" :target="item.target" @click="onItemClick($event, item)" v-ripple
                             :aria-haspopup="item.items != null" :aria-expanded="item === activeItem" role="menuitem" :tabindex="disabled(item) ? null : '0'">
-                            <span :class="['p-menuitem-icon', item.icon]"></span>
+                            <span :class="['p-menuitem-icon', item.icon]" v-if="item.icon"></span>
                             <span class="p-menuitem-text">{{label(item)}}</span>
                             <span class="p-submenu-icon pi pi-angle-right" v-if="item.items"></span>
                         </a>
diff --git a/src/components/menu/Menuitem.vue b/src/components/menu/Menuitem.vue
index 613bb56d0..93d45a9d5 100755
--- a/src/components/menu/Menuitem.vue
+++ b/src/components/menu/Menuitem.vue
@@ -3,12 +3,12 @@
         <template v-if="!template">
             <router-link v-if="item.to && !disabled(item)" :to="item.to" custom v-slot="{navigate, href, isActive, isExactActive}">
                 <a :href="href" @click="onClick($event, navigate)" :class="linkClass(item, {isActive, isExactActive})" v-ripple role="menuitem">
-                    <span :class="['p-menuitem-icon', item.icon]"></span>
+                    <span :class="['p-menuitem-icon', item.icon]" v-if="item.icon"></span>
                     <span class="p-menuitem-text">{{label()}}</span>
                 </a>
             </router-link>
             <a v-else :href="item.url" :class="linkClass(item)" @click="onClick" :target="item.target" role="menuitem" :tabindex="disabled(item) ? null : '0'" v-ripple>
-                <span :class="['p-menuitem-icon', item.icon]"></span>
+                <span :class="['p-menuitem-icon', item.icon]" v-if="item.icon"></span>
                 <span class="p-menuitem-text">{{label()}}</span>
             </a>
         </template>
diff --git a/src/components/menubar/MenubarSub.vue b/src/components/menubar/MenubarSub.vue
index 0c0167e57..4f8631732 100755
--- a/src/components/menubar/MenubarSub.vue
+++ b/src/components/menubar/MenubarSub.vue
@@ -6,13 +6,13 @@
                 <template v-if="!template">
                     <router-link v-if="item.to && !disabled(item)" :to="item.to" custom v-slot="{navigate, href, isActive, isExactActive}">
                         <a :href="href" @click="onItemClick($event, item, navigate)" :class="linkClass(item, {isActive, isExactActive})" v-ripple @keydown="onItemKeyDown($event, item)" role="menuitem">
-                            <span :class="['p-menuitem-icon', item.icon]"></span>
+                            <span :class="['p-menuitem-icon', item.icon]" v-if="item.icon"></span>
                             <span class="p-menuitem-text">{{label(item)}}</span>
                         </a>
                     </router-link>
                     <a v-else :href="item.url" :class="linkClass(item)" :target="item.target" :aria-haspopup="item.items != null" :aria-expanded="item === activeItem"
                         @click="onItemClick($event, item)" @keydown="onItemKeyDown($event, item)" role="menuitem" :tabindex="disabled(item) ? null : '0'" v-ripple>
-                        <span :class="['p-menuitem-icon', item.icon]"></span>
+                        <span :class="['p-menuitem-icon', item.icon]" v-if="item.icon"></span>
                         <span class="p-menuitem-text">{{label(item)}}</span>
                         <span :class="getSubmenuIcon()" v-if="item.items"></span>
                     </a>
diff --git a/src/components/tieredmenu/TieredMenuSub.vue b/src/components/tieredmenu/TieredMenuSub.vue
index d40612584..17361a659 100755
--- a/src/components/tieredmenu/TieredMenuSub.vue
+++ b/src/components/tieredmenu/TieredMenuSub.vue
@@ -6,13 +6,13 @@
                 <template v-if="!template">
                     <router-link v-if="item.to && !disabled(item)" :to="item.to" custom v-slot="{navigate, href, isActive, isExactActive}">
                         <a :href="href" @click="onItemClick($event, item, navigate)" :class="linkClass(item, {isActive, isExactActive})" v-ripple @keydown="onItemKeyDown($event, item)" role="menuitem">
-                            <span :class="['p-menuitem-icon', item.icon]"></span>
+                            <span :class="['p-menuitem-icon', item.icon]" v-if="item.icon"></span>
                             <span class="p-menuitem-text">{{label(item)}}</span>
                         </a>
                     </router-link>
                     <a v-else :href="item.url" :class="linkClass(item)" :target="item.target" :aria-haspopup="item.items != null" :aria-expanded="item === activeItem"
                         @click="onItemClick($event, item)" @keydown="onItemKeyDown($event, item)" role="menuitem" :tabindex="disabled(item) ? null : '0'" v-ripple>
-                        <span :class="['p-menuitem-icon', item.icon]"></span>
+                        <span :class="['p-menuitem-icon', item.icon]" v-if="item.icon"></span>
                         <span class="p-menuitem-text">{{label(item)}}</span>
                         <span class="p-submenu-icon pi pi-angle-right" v-if="item.items"></span>
                     </a>