Skip to content

Commit 88e3ea2

Browse files
committed
remove fullscreen/no-navigation and move/remove _variables
Signed-off-by: John Cowen <john.cowen@konghq.com>
1 parent b707915 commit 88e3ea2

File tree

5 files changed

+10
-61
lines changed

5 files changed

+10
-61
lines changed

packages/kuma-gui/src/app/application/components/app-view/AppView.vue

Lines changed: 1 addition & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -71,20 +71,14 @@
7171
/>
7272
</nav>
7373

74-
<section
75-
:class="{
76-
'is-fullscreen': props.fullscreen,
77-
}"
78-
>
74+
<section>
7975
<XLayout
8076
type="stack"
8177
>
8278
<header
8379
v-if="slots.title || slots.actions"
8480
class="app-view-title-bar"
8581
>
86-
<KongIcon v-if="props.fullscreen" />
87-
8882
<template
8983
v-if="summary.length > 0"
9084
>
@@ -143,7 +137,6 @@
143137
</template>
144138

145139
<script lang="ts" setup>
146-
import { KongIcon } from '@kong/icons'
147140
import { ROUTE_VIEW_PARENT } from '@kumahq/routing/vue'
148141
import { nextTick , provide, inject, watch, ref, onBeforeUnmount , useId } from 'vue'
149142
@@ -162,12 +155,10 @@ type Breadcrumbs = Map<symbol, BreadcrumbItem[]>
162155
163156
const props = withDefaults(defineProps<{
164157
breadcrumbs?: BreadcrumbItem[] | null
165-
fullscreen?: boolean
166158
docs?: string
167159
notifications?: boolean
168160
}>(), {
169161
breadcrumbs: null,
170-
fullscreen: false,
171162
docs: '',
172163
notifications: false,
173164
})
@@ -238,18 +229,6 @@ onBeforeUnmount(() => {
238229
position: relative;
239230
left: -3px;
240231
}
241-
.is-fullscreen {
242-
.app-view-title-bar {
243-
padding: var(--x-space-80) var(--x-space-90);
244-
border-bottom: var(--x-border-width-10) solid var(--x-color-border);
245-
}
246-
.app-view-title-bar h1 {
247-
margin-left: var(--x-space-40);
248-
padding-left: var(--x-space-40);
249-
border-left: var(--x-border-width-10) solid var(--x-color-border);
250-
font-size: var(--x-font-size-60);
251-
}
252-
}
253232
.k-tabs + .route-view > .app-view .app-view-title-bar {
254233
margin-bottom: 20px;
255234
}

packages/kuma-gui/src/app/kuma/components/ApplicationShell.vue

Lines changed: 8 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -162,29 +162,17 @@ const env = useEnv()
162162
const { t } = useI18n()
163163
164164
</script>
165-
<style lang="scss">
166-
html.is-fullscreen {
167-
.application-shell > header {
168-
display: none;
169-
}
170-
.application-shell > .app-content-container > nav {
171-
display: none;
172-
}
173-
.application-shell > .app-content-container > .app-notifications {
174-
display: none;
175-
}
176-
}
177-
html.no-navigation {
178-
.application-shell > .app-content-container .app-sidebar > nav {
179-
display: none;
180-
}
181-
}
182-
</style>
183-
184165
<style lang="scss" scoped>
166+
.application-shell {
167+
--AppHeaderHeight: 60px;
168+
--AppSidebarWidth: 240px;
169+
--AppContentPadding: var(--x-space-80);
170+
--x-drawer-offset-top: var(--AppHeaderHeight);
171+
172+
}
185173
.app-content-container {
174+
display: grid;
186175
padding-top: var(--AppHeaderHeight, initial);
187-
display: var(--AppDisplay);
188176
// Note: `minmax(0, 1fr)` is used because `1fr` implies `minmax(auto, 1fr)` which will allow grid items to grow beyond their container's size.
189177
grid-template-columns: var(--AppSidebarWidth) minmax(0, 1fr);
190178
}

packages/kuma-gui/src/assets/styles/_variables.scss

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

packages/kuma-gui/src/assets/styles/main.scss

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
@use "mixins";
22
@use "fonts";
3-
@use "variables";
43
@use "base";
54
@use "components";
65

packages/x/src/components/x-drawer/XDrawer.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
:has-overlay="false"
77
visible
88
:max-width="props.width"
9-
offset-top="var(--app-slideout-offset-top, 0)"
9+
offset-top="var(--x-drawer-offset-top, 0)"
1010
data-testid="summary"
1111
@close="emit('close')"
1212
>

0 commit comments

Comments
 (0)