Skip to content
This repository was archived by the owner on Feb 6, 2024. It is now read-only.

Commit ae6587f

Browse files
feat: remove deck settings modal
1 parent 34d300c commit ae6587f

File tree

6 files changed

+7
-173
lines changed

6 files changed

+7
-173
lines changed

studio/src/app/modals/editor/app-deck-settings/app-deck-settings.scss

Lines changed: 0 additions & 5 deletions
This file was deleted.

studio/src/app/modals/editor/app-deck-settings/app-deck-settings.tsx

Lines changed: 0 additions & 127 deletions
This file was deleted.

studio/src/app/pages/editor/app-editor/app-editor.tsx

Lines changed: 4 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -3,24 +3,22 @@ import {OverlayEventDetail} from '@ionic/core';
33

44
import {filter, take} from 'rxjs/operators';
55

6-
7-
import {CreateSlidesUtils} from '../../../utils/editor/create-slides.utils';
8-
96
import {AuthUser} from '../../../models/auth-user';
107
import {Slide, SlideTemplate} from '../../../models/slide';
118
import {Deck} from '../../../models/deck';
129

10+
import {CreateSlidesUtils} from '../../../utils/editor/create-slides.utils';
1311
import {ParseStyleUtils} from '../../../utils/editor/parse-style.utils';
1412

1513
import {DeckEventsHandler} from '../../../handlers/editor/deck-events/deck-events.handler';
1614

15+
import {EditorHelper} from '../../../helpers/editor/editor.helper';
16+
1717
import {AuthService} from '../../../services/auth/auth.service';
1818
import {GuestService} from '../../../services/guest/guest.service';
1919
import {NavDirection, NavService} from '../../../services/nav/nav.service';
20-
21-
import {EditorHelper} from '../../../helpers/editor/editor.helper';
22-
import {DeckAction} from '../../../popovers/editor/app-deck-actions/deck-action';
2320
import {DeckEditorService} from '../../../services/deck/deck-editor.service';
21+
import {DeckAction} from '../../../popovers/editor/app-deck-actions/deck-action';
2422

2523
@Component({
2624
tag: 'app-editor',
@@ -545,27 +543,13 @@ export class AppEditor {
545543
await this.toggleFullScreen();
546544
} else if (detail.data.action === DeckAction.JUMP_TO) {
547545
await this.openSlideNavigate();
548-
} else if (detail.data.action === DeckAction.SETTINGS) {
549-
await this.openDeckSettings();
550546
}
551547
}
552548
});
553549

554550
await popover.present();
555551
}
556552

557-
async openDeckSettings() {
558-
const modal: HTMLIonModalElement = await this.modalController.create({
559-
component: 'app-deck-settings'
560-
});
561-
562-
modal.onDidDismiss().then(async (_detail: OverlayEventDetail) => {
563-
// Nothing special
564-
});
565-
566-
await modal.present();
567-
}
568-
569553
render() {
570554
return [
571555
<app-navigation publish={true}></app-navigation>,
@@ -605,11 +589,6 @@ export class AppEditor {
605589
<ion-label>Fullscreen</ion-label>
606590
</ion-tab-button>
607591

608-
<ion-tab-button onClick={() => this.openDeckSettings()} color="primary" class="wider-devices">
609-
<ion-icon name="settings"></ion-icon>
610-
<ion-label>Settings</ion-label>
611-
</ion-tab-button>
612-
613592
<ion-tab-button onClick={(e: UIEvent) => this.openDeckActions(e)} color="primary" class="small-devices">
614593
<ion-icon md="md-more" ios="md-more"></ion-icon>
615594
<ion-label>More</ion-label>

studio/src/app/popovers/editor/app-deck-actions/app-deck-actions.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,10 +17,11 @@ export class AppDeckActions {
1717
}
1818

1919
render() {
20+
// TODO: Share when published
21+
2022
return <div padding>
2123
<a onClick={() => this.closePopover(DeckAction.JUMP_TO)}><p>Jump to slide</p></a>
2224
<a onClick={() => this.closePopover(DeckAction.FULLSCREEN)}><p>Fullscreen</p></a>
23-
<a onClick={() => this.closePopover(DeckAction.SETTINGS)}><p>Settings</p></a>
2425
</div>
2526
}
2627
}
Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
export enum DeckAction {
2-
SETTINGS,
32
FULLSCREEN,
43
JUMP_TO
54
}

studio/src/components.d.ts

Lines changed: 1 addition & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,9 @@
88
import '@stencil/core';
99

1010
import '@ionic/core';
11+
import 'ionicons';
1112
import 'deckdeckgo';
1213
import 'deckdeckgo-inline-editor';
13-
import 'ionicons';
1414
import {
1515
EventEmitter,
1616
} from '@stencil/core';
@@ -114,9 +114,6 @@ export namespace Components {
114114
interface AppFeedLazyImg {}
115115
interface AppFeedLazyImgAttributes extends StencilHTMLAttributes {}
116116

117-
interface AppDeckSettings {}
118-
interface AppDeckSettingsAttributes extends StencilHTMLAttributes {}
119-
120117
interface AppGif {}
121118
interface AppGifAttributes extends StencilHTMLAttributes {}
122119

@@ -210,7 +207,6 @@ declare global {
210207
'AppPopular': Components.AppPopular;
211208
'AppFeedCard': Components.AppFeedCard;
212209
'AppFeedLazyImg': Components.AppFeedLazyImg;
213-
'AppDeckSettings': Components.AppDeckSettings;
214210
'AppGif': Components.AppGif;
215211
'AppPublish': Components.AppPublish;
216212
'AppSlideNavigate': Components.AppSlideNavigate;
@@ -246,7 +242,6 @@ declare global {
246242
'app-popular': Components.AppPopularAttributes;
247243
'app-feed-card': Components.AppFeedCardAttributes;
248244
'app-feed-lazy-img': Components.AppFeedLazyImgAttributes;
249-
'app-deck-settings': Components.AppDeckSettingsAttributes;
250245
'app-gif': Components.AppGifAttributes;
251246
'app-publish': Components.AppPublishAttributes;
252247
'app-slide-navigate': Components.AppSlideNavigateAttributes;
@@ -352,12 +347,6 @@ declare global {
352347
new (): HTMLAppFeedLazyImgElement;
353348
};
354349

355-
interface HTMLAppDeckSettingsElement extends Components.AppDeckSettings, HTMLStencilElement {}
356-
var HTMLAppDeckSettingsElement: {
357-
prototype: HTMLAppDeckSettingsElement;
358-
new (): HTMLAppDeckSettingsElement;
359-
};
360-
361350
interface HTMLAppGifElement extends Components.AppGif, HTMLStencilElement {}
362351
var HTMLAppGifElement: {
363352
prototype: HTMLAppGifElement;
@@ -481,7 +470,6 @@ declare global {
481470
'app-popular': HTMLAppPopularElement
482471
'app-feed-card': HTMLAppFeedCardElement
483472
'app-feed-lazy-img': HTMLAppFeedLazyImgElement
484-
'app-deck-settings': HTMLAppDeckSettingsElement
485473
'app-gif': HTMLAppGifElement
486474
'app-publish': HTMLAppPublishElement
487475
'app-slide-navigate': HTMLAppSlideNavigateElement
@@ -517,7 +505,6 @@ declare global {
517505
'app-popular': HTMLAppPopularElement;
518506
'app-feed-card': HTMLAppFeedCardElement;
519507
'app-feed-lazy-img': HTMLAppFeedLazyImgElement;
520-
'app-deck-settings': HTMLAppDeckSettingsElement;
521508
'app-gif': HTMLAppGifElement;
522509
'app-publish': HTMLAppPublishElement;
523510
'app-slide-navigate': HTMLAppSlideNavigateElement;

0 commit comments

Comments
 (0)