Skip to content

Commit a810ac2

Browse files
committed
fix: macos + needs shift to be accessible
so we allow the = to be used a zoomin
1 parent 0dcb507 commit a810ac2

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

ts/hooks/useZoomingShortcut.ts

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
import useKey from 'react-use/lib/useKey';
22
import { ZOOM_FACTOR } from '../session/constants';
3+
import { isMacOS } from '../OS';
34

45
const changeZoom = async (
56
change: { typeOfChange: 'delta'; delta: number } | { typeOfChange: 'reset' }
@@ -20,15 +21,17 @@ const changeZoom = async (
2021

2122
export function useZoomShortcuts() {
2223
useKey(
23-
event => (event.ctrlKey || event.metaKey) && (event.key === '+' || event.key === '='),
24+
event =>
25+
// macos + is accessible through shift only, so we need to allow `=` for macos
26+
(event.ctrlKey || event.metaKey) && (event.key === '+' || (isMacOS() && event.key === '=')),
2427
event => {
2528
event.preventDefault();
2629
void changeZoom({ typeOfChange: 'delta', delta: ZOOM_FACTOR.STEP });
2730
}
2831
);
2932

3033
useKey(
31-
event => (event.ctrlKey || event.metaKey) && event.key === '-',
34+
event => (event.ctrlKey || event.metaKey) && event.key === '-', // macos - is accessible without shift
3235
event => {
3336
event.preventDefault();
3437
void changeZoom({ typeOfChange: 'delta', delta: -ZOOM_FACTOR.STEP });

0 commit comments

Comments
 (0)