refactor(editor): unify selection key + clean node view
This commit is contained in:
parent
e296fc0979
commit
5ff0c22724
@ -436,18 +436,18 @@ function NodeView(props: {
|
||||
boxSizing: 'border-box',
|
||||
padding: `${node.props.paddingY ?? 0}px ${node.props.paddingX ?? 0}px`,
|
||||
overflow: 'hidden',
|
||||
}}
|
||||
>
|
||||
<span
|
||||
style={{
|
||||
whiteSpace: 'pre-wrap',
|
||||
fontSize: node.props.fontSize ?? 24,
|
||||
color: node.props.color ?? '#fff',
|
||||
fontWeight: node.props.fontWeight ?? 400,
|
||||
letterSpacing: `${node.props.letterSpacing ?? 0}px`,
|
||||
writingMode: writingModeStyle,
|
||||
cursor: node.props.link ? 'pointer' : 'default',
|
||||
}}
|
||||
>
|
||||
<span
|
||||
style={{
|
||||
whiteSpace: 'pre-wrap',
|
||||
fontSize: node.props.fontSize ?? 24,
|
||||
color: node.props.color ?? '#fff',
|
||||
fontWeight: node.props.fontWeight ?? 400,
|
||||
letterSpacing: `${node.props.letterSpacing ?? 0}px`,
|
||||
writingMode: writingModeStyle,
|
||||
cursor: node.props.link ? 'pointer' : 'default',
|
||||
}}
|
||||
onClick={() => {
|
||||
if (!node.props.link) return;
|
||||
const head = node.props.linkHead ?? 'http://';
|
||||
|
||||
@ -14,6 +14,7 @@ import { bindEditorHotkeys } from './hotkeys';
|
||||
import { Canvas } from './Canvas';
|
||||
import { ContextMenu, type ContextMenuState } from './ContextMenu';
|
||||
import { Inspector } from './Inspector';
|
||||
import { selectionKeyOf } from './selection';
|
||||
import { createInitialState, editorReducer, exportScreenJSON } from './store';
|
||||
|
||||
const { Header, Sider, Content } = Layout;
|
||||
@ -86,10 +87,7 @@ export function EditorApp() {
|
||||
|
||||
const closeContextMenu = useCallback(() => setCtxMenu(null), []);
|
||||
|
||||
const selectionKeyOf = useCallback((ids: string[]) => {
|
||||
// Keep stable regardless of ordering.
|
||||
return [...ids].sort().join('|');
|
||||
}, []);
|
||||
// selectionKeyOf imported from ./selection
|
||||
|
||||
const ctxMenuSyncedRef = useRef(false);
|
||||
|
||||
@ -103,7 +101,7 @@ export function EditorApp() {
|
||||
}
|
||||
dispatch(action);
|
||||
},
|
||||
[ctxMenu, dispatch, selectionKeyOf, state.selection.ids],
|
||||
[ctxMenu, dispatch, state.selection.ids],
|
||||
);
|
||||
|
||||
useEffect(() => {
|
||||
|
||||
Loading…
Reference in New Issue
Block a user