Bugzilla – Attachment 176971 Details for
Bug 38854
Unused 'class' prop in ToolbarButton
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 38854: Remove 'class' property from ToolbarButton
Bug-38854-Remove-class-property-from-ToolbarButton.patch (text/plain), 6.80 KB, created by
Matt Blenkinsop
on 2025-01-23 14:39:43 UTC
(
hide
)
Description:
Bug 38854: Remove 'class' property from ToolbarButton
Filename:
MIME Type:
Creator:
Matt Blenkinsop
Created:
2025-01-23 14:39:43 UTC
Size:
6.80 KB
patch
obsolete
>From 7c5cedb08f4a3bdc8af301075809f01d67daabeb Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >Date: Thu, 9 Jan 2025 14:30:09 +0100 >Subject: [PATCH] Bug 38854: Remove 'class' property from ToolbarButton > >We never use another value than the default one. >We can simply remove this property. > >Test plan: >Rebuild the dist files using `yarn js:build` and confirm that the >buttons in the toolbar of the ERM and Preservation modules still appears >and are styled like before. > >Signed-off-by: William Lavoie <william.lavoie@inLibro.com> >Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> >--- > .../prog/js/vue/components/ERM/AgreementsShow.vue | 1 - > .../js/vue/components/ERM/EHoldingsLocalPackagesShow.vue | 1 - > .../prog/js/vue/components/ERM/EHoldingsLocalTitlesShow.vue | 1 - > .../prog/js/vue/components/ERM/LicensesShow.vue | 1 - > .../vue/components/ERM/UsageStatisticsDataProvidersShow.vue | 1 - > .../vue/components/Preservation/SettingsProcessingsShow.vue | 1 - > .../prog/js/vue/components/Preservation/TrainsShow.vue | 2 -- > .../intranet-tmpl/prog/js/vue/components/ToolbarButton.vue | 6 +----- > 8 files changed, 1 insertion(+), 13 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/AgreementsShow.vue b/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/AgreementsShow.vue >index c8330cddaa0..ad6050e2196 100644 >--- a/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/AgreementsShow.vue >+++ b/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/AgreementsShow.vue >@@ -7,7 +7,6 @@ > name: 'AgreementsFormAddEdit', > params: { agreement_id: agreement.agreement_id }, > }" >- class="btn btn-default" > icon="pencil" > :title="$__('Edit')" > /> >diff --git a/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/EHoldingsLocalPackagesShow.vue b/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/EHoldingsLocalPackagesShow.vue >index 5f73b9d80d4..8c949f54ae7 100644 >--- a/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/EHoldingsLocalPackagesShow.vue >+++ b/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/EHoldingsLocalPackagesShow.vue >@@ -7,7 +7,6 @@ > name: 'EHoldingsLocalPackagesFormAddEdit', > params: { package_id: erm_package.package_id }, > }" >- class="btn btn-default" > icon="pencil" > :title="$__('Edit')" > /> >diff --git a/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/EHoldingsLocalTitlesShow.vue b/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/EHoldingsLocalTitlesShow.vue >index c11003a6a31..58eedaa1d1a 100644 >--- a/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/EHoldingsLocalTitlesShow.vue >+++ b/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/EHoldingsLocalTitlesShow.vue >@@ -7,7 +7,6 @@ > name: 'EHoldingsLocalTitlesFormAddEdit', > params: { title_id: title.title_id }, > }" >- class="btn btn-default" > icon="pencil" > :title="$__('Edit')" > /> >diff --git a/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/LicensesShow.vue b/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/LicensesShow.vue >index 799afe24ea3..6e91f7d59bf 100644 >--- a/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/LicensesShow.vue >+++ b/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/LicensesShow.vue >@@ -7,7 +7,6 @@ > name: 'LicensesFormAddEdit', > params: { license_id: license.license_id }, > }" >- class="btn btn-default" > icon="pencil" > :title="$__('Edit')" > /> >diff --git a/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/UsageStatisticsDataProvidersShow.vue b/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/UsageStatisticsDataProvidersShow.vue >index 21d46e2e700..796db87c0af 100644 >--- a/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/UsageStatisticsDataProvidersShow.vue >+++ b/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/UsageStatisticsDataProvidersShow.vue >@@ -10,7 +10,6 @@ > usage_data_provider.erm_usage_data_provider_id, > }, > }" >- class="btn btn-default" > icon="pencil" > :title="$__('Edit')" > /> >diff --git a/koha-tmpl/intranet-tmpl/prog/js/vue/components/Preservation/SettingsProcessingsShow.vue b/koha-tmpl/intranet-tmpl/prog/js/vue/components/Preservation/SettingsProcessingsShow.vue >index 82c35eeaf2b..4c136203089 100644 >--- a/koha-tmpl/intranet-tmpl/prog/js/vue/components/Preservation/SettingsProcessingsShow.vue >+++ b/koha-tmpl/intranet-tmpl/prog/js/vue/components/Preservation/SettingsProcessingsShow.vue >@@ -7,7 +7,6 @@ > name: 'SettingsProcessingsFormEdit', > params: { processing_id: processing.processing_id }, > }" >- class="btn btn-default" > icon="pencil" > :title="$__('Edit')" > /> >diff --git a/koha-tmpl/intranet-tmpl/prog/js/vue/components/Preservation/TrainsShow.vue b/koha-tmpl/intranet-tmpl/prog/js/vue/components/Preservation/TrainsShow.vue >index 9b3496b09d5..43d369422d0 100644 >--- a/koha-tmpl/intranet-tmpl/prog/js/vue/components/Preservation/TrainsShow.vue >+++ b/koha-tmpl/intranet-tmpl/prog/js/vue/components/Preservation/TrainsShow.vue >@@ -79,7 +79,6 @@ > name: 'TrainsFormAddItem', > params: { train_id: train.train_id }, > }" >- class="btn btn-default" > icon="plus" > :title="$__('Add items')" > /> >@@ -96,7 +95,6 @@ > name: 'TrainsFormEdit', > params: { train_id: train.train_id }, > }" >- class="btn btn-default" > icon="pencil" > :title="$__('Edit')" > /> >diff --git a/koha-tmpl/intranet-tmpl/prog/js/vue/components/ToolbarButton.vue b/koha-tmpl/intranet-tmpl/prog/js/vue/components/ToolbarButton.vue >index 4cfada715cc..f44fb9d3cdc 100644 >--- a/koha-tmpl/intranet-tmpl/prog/js/vue/components/ToolbarButton.vue >+++ b/koha-tmpl/intranet-tmpl/prog/js/vue/components/ToolbarButton.vue >@@ -1,5 +1,5 @@ > <template> >- <router-link :to="to" :class="class" >+ <router-link :to="to" class="btn btn-default" > ><font-awesome-icon v-if="icon" :icon="icon" /> {{ title }}</router-link > > > </template> >@@ -10,10 +10,6 @@ export default { > to: { > type: [String, Object], > }, >- class: { >- type: String, >- default: "btn btn-default", >- }, > icon: { > type: String, > required: false, >-- >2.48.1
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 38854
:
176292
|
176450
|
176457
|
176903
| 176971