Merge pull request #2044 from omnivore-app/fix/web-keyboard-modal-width

Better minWidth for small screens
This commit is contained in:
Jackson Harper 2023-04-14 12:47:06 +08:00 committed by GitHub
commit 6bfe56b1c9
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 6 additions and 6 deletions

View file

@ -6,11 +6,8 @@ import {
ModalTitleBar,
} from '../elements/ModalPrimitives'
import type { KeyboardCommand } from '../../lib/keyboardShortcuts/useKeyboardShortcuts'
import { HStack, VStack, Box } from '../elements/LayoutPrimitives'
import { VStack, Box } from '../elements/LayoutPrimitives'
import { StyledText } from '../elements/StyledText'
import { Button } from '../elements/Button'
import { CrossIcon } from '../elements/images/CrossIcon'
import { theme } from '../tokens/stitches.config'
import {
navigationCommands,
searchBarCommands,
@ -143,6 +140,9 @@ export function KeyboardShortcutListModal(
minWidth: '650px',
minHeight: '430px',
height: '430px',
'@mdDown': {
minWidth: '320px',
},
}}
onInteractOutside={() => {
// remove focus from modal
@ -163,11 +163,12 @@ export function KeyboardShortcutListModal(
onOpenChange={props.onOpenChange}
/>
<VStack
id="keyboard-shortcuts-ctr"
distribution="start"
css={{
width: '100%',
maxHeight: '100%',
overflow: 'scroll',
overflowY: 'scroll',
}}
>
<ShortcutListSection

View file

@ -113,7 +113,6 @@ export default function Home(): JSX.Element {
}
break
case 'mark-read':
console.log('marking read: ', article)
if (article) {
articleReadingProgressMutation({
id: article.id,