Removing "print PDF" button from desktop version, same can be accompl…

…ished with the regular print button using the system dialogue
TheRealKANi · Feb 12, 2022 · 30d1670 · 30d1670
1 parent ec16e72
commit 30d1670
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 19 deletions.
diff --git a/td.vue/src/views/Report.vue b/td.vue/src/views/Report.vue
@@ -53,12 +53,6 @@
 
             <b-col class="text-right right">
                 <b-btn-group>
-                    <td-form-button
-                        id="td-save-pdf-btn"
-                        :onBtnClick="noOp"
-                        v-if="isElectron"
-                        icon="file-pdf"
-                        :text="$t('forms.savePdf')" />
                     <td-form-button
                         id="td-print-btn"
                         :onBtnClick="print"
@@ -155,7 +149,6 @@
 <script>
 import { mapState, mapGetters } from 'vuex';
 
-import env from '@/service/env.js';
 import { getProviderType } from '@/service/provider/providers.js';
 import TdCoversheet from '@/components/report/Coversheet.vue';
 import TdDiagramDetail from '@/components/report/DiagramDetail.vue';
@@ -182,8 +175,7 @@ export default {
                 mitigated: true,
                 diagrams: true,
                 branding: true
-            },
-            isElectron: env.isElectron()
+            }
         };
     },
     computed: {
@@ -202,10 +194,6 @@ export default {
         })
     },
     methods: {
-        noOp() {
-            console.log('TODO');
-            return;
-        },
         onCloseClick() {
             this.$router.push({ name: `${this.providerType}ThreatModel`, params: this.$route.params });
         },

diff --git a/td.vue/tests/unit/views/report.spec.js b/td.vue/tests/unit/views/report.spec.js
@@ -105,12 +105,6 @@ describe('views/PrinterReport.vue', () => {
             .toEqual(true);
     });
 
-    it('does nothing with a noop', () => {
-        console.log = jest.fn();
-        wrapper.vm.noOp();
-        expect(console.log).toHaveBeenCalledWith('TODO');
-    });
-
     it('closes the report', () => {
         wrapper.vm.onCloseClick();
         expect(routerMock.push).toHaveBeenCalledWith({