Skip to content

Commit

Permalink
fix: overflow social icons in AppHeaderDialog (#803)
Browse files Browse the repository at this point in the history
  • Loading branch information
bdrtsky committed Feb 1, 2023
1 parent 79675be commit 70658f3
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 7 deletions.
6 changes: 5 additions & 1 deletion components/app/AppHeader.vue
Expand Up @@ -80,7 +80,11 @@ css({
alignItems: 'center',
flex: 'none',
'&.left': {
gridColumn: 'span 4 / span 4'
gridColumn: 'span 4 / span 4',
marginLeft: 'calc(0px - {space.4})',
'@lg': {
marginLeft: 0
},
},
'&.center': {
gridColumn: 'span 4 / span 4',
Expand Down
15 changes: 9 additions & 6 deletions components/app/AppHeaderDialog.vue
@@ -1,6 +1,4 @@
<script setup lang="ts">
import ThemeSelect from './ThemeSelect.vue'
const { navigation } = useContent()
const { config } = useDocus()
Expand All @@ -27,13 +25,12 @@ watch(visible, v => (v ? open() : close()))
<teleport to="body">
<nav v-if="visible" class="dialog" @click="close">
<div @click.stop>
<div>
<div class="wrapper">
<button aria-label="Menu" @click="close">
<Icon name="heroicons-outline:x" aria-hidden="”true”" />
</button>

<div class="icons">
<ThemeSelect />
<AppSocialIcons />
</div>
</div>
Expand All @@ -49,7 +46,8 @@ css({
button: {
position: 'relative',
zIndex: '10',
borderRadius: '{radii.xl}',
display: 'flex',
padding: '{space.4}',
'@lg': {
display: 'none'
},
Expand Down Expand Up @@ -79,6 +77,12 @@ css({
'@lg': {
display: 'none'
},
'.wrapper': {
marginLeft: 'calc(0px - {space.4})',
},
'.icons': {
overflow: 'auto'
},
// Dialog content
'& > div': {
maxWidth: '{size.xs}',
Expand All @@ -104,7 +108,6 @@ css({
'.icons': {
display: 'flex',
alignItems: 'center',
gap: '{space.4}'
}
}
}
Expand Down

0 comments on commit 70658f3

Please sign in to comment.