-
Notifications
You must be signed in to change notification settings - Fork 300
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
フルスクリーンモードを追加(#2251) #2273
base: main
Are you sure you want to change the base?
フルスクリーンモードを追加(#2251) #2273
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -134,6 +134,10 @@ const openHelpDialog = () => { | |
}); | ||
}; | ||
|
||
const toggleFullScreenMode = async () => { | ||
window.backend.toggleFullScreenMode(); | ||
}; | ||
|
||
const createNewProject = async () => { | ||
if (!uiLocked.value) { | ||
await store.dispatch("CREATE_NEW_PROJECT", {}); | ||
|
@@ -396,7 +400,16 @@ const menudata = computed<MenuItemData[]>(() => [ | |
closeAllDialog(); | ||
}, | ||
disableWhenUiLocked: false, | ||
subMenu: [...props.viewSubMenuData], | ||
subMenu: [ | ||
...props.viewSubMenuData, | ||
{ type: "separator" }, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. (細かいですが)トークエディタだと一番上にseparatorが来そう? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ほんとですね!! ちょっと見た目変になりそう。 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. コメントありがとうございます! There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 大丈夫だと思います。 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. たしかに、一旦なしがとりあえずで一番良さそう!ご提案ありがとうございます! |
||
{ | ||
type: "button", | ||
label: "全画面表示/ウィンドウ表示切り替え", | ||
onClick: toggleFullScreenMode, | ||
disableWhenUiLocked: false, | ||
}, | ||
], | ||
}, | ||
{ | ||
type: "root", | ||
|
@@ -527,6 +540,10 @@ function registerHotkeyForAllEditors(action: Omit<HotkeyAction, "editor">) { | |
}); | ||
} | ||
|
||
registerHotkeyForAllEditors({ | ||
callback: toggleFullScreenMode, | ||
name: "全画面表示/ウィンドウ表示切り替え", | ||
}); | ||
registerHotkeyForAllEditors({ | ||
callback: createNewProject, | ||
name: "新規プロジェクト", | ||
|
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -146,6 +146,10 @@ export const defaultHotkeySettings: HotkeySettingType[] = [ | |||||
action: "新規プロジェクト", | ||||||
combination: HotkeyCombination(!isMac ? "Ctrl N" : "Meta N"), | ||||||
}, | ||||||
{ | ||||||
action: "全画面表示/ウィンドウ表示切り替え", | ||||||
combination: HotkeyCombination(!isMac ? "F11" : "Ctrl Meta F"), | ||||||
}, | ||||||
{ | ||||||
action: "プロジェクトを名前を付けて保存", | ||||||
combination: HotkeyCombination(!isMac ? "Ctrl Shift S" : "Shift Meta S"), | ||||||
|
@@ -273,6 +277,7 @@ export interface Sandbox { | |||||
closeWindow(): void; | ||||||
minimizeWindow(): void; | ||||||
maximizeWindow(): void; | ||||||
toggleFullScreenMode(): void; | ||||||
logError(...params: unknown[]): void; | ||||||
logWarn(...params: unknown[]): void; | ||||||
logInfo(...params: unknown[]): void; | ||||||
|
@@ -466,6 +471,7 @@ export const hotkeyActionNameSchema = z.enum([ | |||||
"元に戻す", | ||||||
"やり直す", | ||||||
"新規プロジェクト", | ||||||
"全画面表示/ウィンドウ表示切り替え", | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 丁寧な表記いいですね!
Suggested change
(slackが「フルスクリーン表示の切り替え」だったので真似してみました) |
||||||
"プロジェクトを名前を付けて保存", | ||||||
"プロジェクトを上書き保存", | ||||||
"プロジェクトを読み込む", | ||||||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
の方がいいかも?(これを変えるなら他のところも変える必要があるはず)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
たしかに
FULLSCREENMODE
一単語はほんのちょっぴり違和感あるかもですね!TOGGLE_FULLSCREEN_MODE
という手もありそう。