fix: Editor mode shortcut keys not working #10515
Conversation
|
|
||
| const isPrinting = usePrintMode(); | ||
|
|
||
| const isPrinting = usePrintMode(); |
|
|
||
| const EditPage = (props) => { | ||
| const isEditable = useIsEditable(); | ||
| const { mutate: mutateEditorMode } = useEditorMode(); |
There was a problem hiding this comment.
useEditorMode は mutate を提供していない
| import type { JSX, ReactNode } from 'react'; | ||
|
|
||
| // biome-ignore lint/style/noRestrictedImports: ignore | ||
| import { usePrintMode } from '~/client/services/use-print-mode'; |
There was a problem hiding this comment.
usePrintMode は最近実装された hook だと思うのですが Lint error が出ていました。
一旦 ignore させ CI 上でエラーを出さないようにしたのですがどのように対処するのがよいでしょうか？
There was a problem hiding this comment.
なるほど、悩ましい
やりたいこととしては、print mode だったらサイドコンテンツがないときと同じように children のみをレンダリングさせたいんだけど、このままだと SSR でも import してしまうのであまりよくない
かといってプログラマブルに制御するのをやめて、 div.grw-side-contents-container に d-print-none を設定して CSS 側で消すことにすると、そもそもの #10505 が直らない
(と、コード内コメントに書いておいた方がいいな)
| import type { JSX, ReactNode } from 'react'; | ||
|
|
||
| // biome-ignore lint/style/noRestrictedImports: ignore | ||
| import { usePrintMode } from '~/client/services/use-print-mode'; |
There was a problem hiding this comment.
なるほど、悩ましい
やりたいこととしては、print mode だったらサイドコンテンツがないときと同じように children のみをレンダリングさせたいんだけど、このままだと SSR でも import してしまうのであまりよくない
かといってプログラマブルに制御するのをやめて、 div.grw-side-contents-container に d-print-none を設定して CSS 側で消すことにすると、そもそもの #10505 が直らない
(と、コード内コメントに書いておいた方がいいな)
| import type { JSX, ReactNode } from 'react'; | ||
|
|
||
| // biome-ignore lint/style/noRestrictedImports: ignore | ||
| import { usePrintMode } from '~/client/services/use-print-mode'; |
yuki-takei
deleted the
fix/174327-editor-mode-shortcut-keys-not-working-2
branch
|
This pull request has been removed from the queue for the following reason: The draft pull request cannot be created. If you want to requeue this pull request, you can post a |
Task
https://redmine.weseek.co.jp/issues/174327