Skip to content

Commit 6d6c2db

Browse files
rohityadavcloudDaanHoogland
authored andcommitted
ui: Allow font-awesome icon usage and optimise icon size inconsistency (apache#9744)
Signed-off-by: Rohit Yadav <[email protected]> Co-authored-by: dahn <[email protected]>
1 parent d78074b commit 6d6c2db

File tree

4 files changed

+31
-15
lines changed

4 files changed

+31
-15
lines changed

ui/src/components/menu/SMenu.vue

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,12 @@
5858
v-if="item.meta.icon && typeof (item.meta.icon) === 'string'"
5959
:icon="item.meta.icon"
6060
@click="() => { handleClickParentMenu(item) }" />
61+
<font-awesome-icon
62+
v-else-if="item.meta.icon && Array.isArray(item.meta.icon)"
63+
:icon="item.meta.icon"
64+
class="anticon"
65+
:style="[$store.getters.darkMode ? { color: 'rgba(255, 255, 255, 0.65)' } : { color: '#888' }]"
66+
@click="() => { handleClickParentMenu(item) }" />
6167
<span>{{ $t(item.meta.title) }}</span>
6268
</router-link>
6369
</a-menu-item>

ui/src/components/view/InfoCard.vue

Lines changed: 20 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -34,12 +34,12 @@
3434
<resource-icon :image="getImage(resource.icon && resource.icon.base64image || images.template || images.iso || resourceIcon)" size="4x" style="margin-right: 5px"/>
3535
</span>
3636
<span v-else>
37-
<os-logo v-if="resource.ostypeid || resource.ostypename" :osId="resource.ostypeid" :osName="resource.ostypename" size="4x" @update-osname="setResourceOsType"/>
37+
<os-logo v-if="resource.ostypeid || resource.ostypename" :osId="resource.ostypeid" :osName="resource.ostypename" size="3x" @update-osname="setResourceOsType"/>
3838
<render-icon v-else-if="typeof $route.meta.icon ==='string'" style="font-size: 36px" :icon="$route.meta.icon" />
3939
<font-awesome-icon
4040
v-else-if="$route.meta.icon && Array.isArray($route.meta.icon)"
4141
:icon="$route.meta.icon"
42-
size="4x"
42+
size="3x"
4343
class="anticon"
4444
:style="[$store.getters.darkMode ? { color: 'rgba(255, 255, 255, 0.65)' } : { color: '#888' }]" />
4545
<render-icon v-else style="font-size: 36px" :svgIcon="$route.meta.icon" />
@@ -196,7 +196,10 @@
196196
<div class="resource-detail-item" v-if="('cpunumber' in resource && 'cpuspeed' in resource) || resource.cputotal">
197197
<div class="resource-detail-item__label">{{ $t('label.cpu') }}</div>
198198
<div class="resource-detail-item__details">
199-
<appstore-outlined />
199+
<font-awesome-icon
200+
:icon="['fa-solid', 'fa-microchip']"
201+
class="anticon"
202+
:style="[$store.getters.darkMode ? { color: 'rgba(255, 255, 255, 0.65)' } : { color: '#888' }]" />
200203
<span v-if="'cpunumber' in resource && 'cpuspeed' in resource">{{ resource.cpunumber }} CPU x {{ (resource.cpuspeed / 1000.0).toFixed(2) }} GHz
201204
<a-tooltip placement="top">
202205
<template #title>
@@ -234,7 +237,11 @@
234237
<div class="resource-detail-item" v-if="'memory' in resource">
235238
<div class="resource-detail-item__label">{{ $t('label.memory') }}</div>
236239
<div class="resource-detail-item__details">
237-
<bulb-outlined />{{ resource.memory + ' ' + $t('label.mb.memory') }}
240+
<font-awesome-icon
241+
:icon="['fa-solid', 'fa-memory']"
242+
class="anticon"
243+
:style="[$store.getters.darkMode ? { color: 'rgba(255, 255, 255, 0.65)' } : { color: '#888' }]" />
244+
{{ resource.memory + ' ' + $t('label.mb.memory') }}
238245
</div>
239246
<div>
240247
<span v-if="resource.memorykbs && resource.memoryintusablekbs">
@@ -384,7 +391,10 @@
384391
v-for="(eth, index) in resource.nic"
385392
:key="eth.id"
386393
style="margin-left: -24px; margin-top: 5px;">
387-
<api-outlined />
394+
<font-awesome-icon
395+
:icon="['fa-solid', 'fa-ethernet']"
396+
class="anticon"
397+
:style="[$store.getters.darkMode ? { color: 'rgba(255, 255, 255, 0.65)' } : { color: '#888' }]" />
388398
<strong>eth{{ index }}</strong>&nbsp;
389399
<copy-label :label="eth.ip6address ? eth.ipaddress + ', ' + eth.ip6address : eth.ipaddress" />&nbsp;
390400
<a-tag v-if="eth.isdefault">
@@ -409,7 +419,11 @@
409419
v-for="network in resource.networks"
410420
:key="network.id"
411421
style="margin-top: 5px;">
412-
<api-outlined />{{ network.name }}
422+
<font-awesome-icon
423+
:icon="['fa-solid', 'fa-ethernet']"
424+
class="anticon"
425+
:style="[$store.getters.darkMode ? { color: 'rgba(255, 255, 255, 0.65)' } : { color: '#888' }]" />
426+
{{ network.name }}
413427
<span v-if="resource.defaultnetworkid === network.id">
414428
({{ $t('label.default') }})
415429
</span>

ui/src/components/view/ListView.vue

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@
4444
<span v-if="record.icon && record.icon.base64image">
4545
<resource-icon :image="record.icon.base64image" size="2x"/>
4646
</span>
47-
<os-logo v-else :osId="record.ostypeid" :osName="record.osdisplayname" size="2x" />
47+
<os-logo v-else :osId="record.ostypeid" :osName="record.osdisplayname" size="xl" />
4848
</span>
4949
<span style="min-width: 120px" >
5050
<QuickView
@@ -58,12 +58,12 @@
5858
</span>
5959
<span v-if="$showIcon() && !['vm', 'vnfapp'].includes($route.path.split('/')[1])" style="margin-right: 5px">
6060
<resource-icon v-if="$showIcon() && record.icon && record.icon.base64image" :image="record.icon.base64image" size="2x"/>
61-
<os-logo v-else-if="record.ostypename" :osName="record.ostypename" size="2x" />
61+
<os-logo v-else-if="record.ostypename" :osName="record.ostypename" size="xl" />
6262
<render-icon v-else-if="typeof $route.meta.icon ==='string'" style="font-size: 16px;" :icon="$route.meta.icon"/>
6363
<render-icon v-else style="font-size: 16px;" :svgIcon="$route.meta.icon" />
6464
</span>
6565
<span v-else :style="{ 'margin-right': record.ostypename ? '5px' : '0' }">
66-
<os-logo v-if="record.ostypename" :osName="record.ostypename" size="1x" />
66+
<os-logo v-if="record.ostypename" :osName="record.ostypename" size="xl" />
6767
</span>
6868

6969
<span v-if="record.hasannotations">

ui/src/core/ext.js

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -18,15 +18,11 @@
1818
import { library } from '@fortawesome/fontawesome-svg-core'
1919
import { FontAwesomeIcon } from '@fortawesome/vue-fontawesome'
2020

21-
// import { fab } from '@fortawesome/free-brands-svg-icons'
22-
// import { fas } from '@fortawesome/free-solid-svg-icons'
23-
// import { far } from '@fortawesome/free-regular-svg-icons'
24-
2521
import { faCentos, faUbuntu, faDebian, faSuse, faRedhat, faFedora, faLinux, faFreebsd, faApple, faWindows, faJava } from '@fortawesome/free-brands-svg-icons'
26-
import { faCompactDisc, faCameraRetro, faDharmachakra } from '@fortawesome/free-solid-svg-icons'
22+
import { fas, faCompactDisc, faCameraRetro, faDharmachakra } from '@fortawesome/free-solid-svg-icons'
2723

2824
library.add(faCentos, faUbuntu, faDebian, faSuse, faRedhat, faFedora, faLinux, faFreebsd, faApple, faWindows, faJava)
29-
library.add(faCompactDisc, faCameraRetro, faDharmachakra)
25+
library.add(fas, faCompactDisc, faCameraRetro, faDharmachakra)
3026

3127
export default {
3228
install: (app) => {

0 commit comments

Comments
 (0)