mirror of
https://github.com/webstudio-is/webstudio.git
synced 2026-11-09 05:26:24 +09:00
Ivan Starkov
4753c2ac46
## Description Shared link name is not editable because every input change closes popover with the input. Here is the fix and the explanation. This is already fixed in rc radix versions and will be in the next stable release https://github.com/radix-ui/primitives/blob/28bebf2c6992d056244845c898abeff45dec2871/packages/react/focus-scope/src/FocusScope.tsx#L115 Without this PR the issue looks: Editing input <img width="353" alt="image" src="https://github.com/webstudio-is/webstudio/assets/5077042/b6ba547c-4286-45f2-b351-9122780cb482"> cause this button <img width="216" alt="image" src="https://github.com/webstudio-is/webstudio/assets/5077042/4d6a26c3-3336-4995-b0b4-c38200e34e07"> to switch onto `pending` state <img width="211" alt="image" src="https://github.com/webstudio-is/webstudio/assets/5077042/f214d9a9-3454-4b1b-8603-48224fa370d1"> what cause handleMutations to move focus onto parent popover Child popover lost focus and closes. ## Steps for reproduction Click Share, Edit link name, see it works ## Code Review - [ ] hi @kof, I need you to do - conceptual review (architecture, feature-correctness) - detailed review (read every line) - test it on preview ## Before requesting a review - [ ] made a self-review - [ ] added inline comments where things may be not obvious (the "why", not "what") ## Before merging - [ ] tested locally and on preview environment (preview dev login: 5de6) - [ ] updated [test cases](https://github.com/webstudio-is/webstudio/blob/main/apps/builder/docs/test-cases.md) document - [ ] added tests - [ ] if any new env variables are added, added them to `.env.example` and the `builder/env-check.js` if mandatory
72 lines
2.6 KiB
JSON
72 lines
2.6 KiB
JSON
{
|
|
"packageManager": "pnpm@8.4.0",
|
|
"name": "webstudio",
|
|
"version": "0.0.0",
|
|
"private": true,
|
|
"scripts": {
|
|
"build": "turbo run build",
|
|
"build-packages": "turbo run build --filter=\"!@webstudio-is/builder\"",
|
|
"dts": "turbo run dts",
|
|
"size-test": "turbo run size-test",
|
|
"checks": "turbo run checks",
|
|
"dev": "pnpm run --parallel --recursive dev",
|
|
"lint": "pnpm eslint . --ext .ts,.tsx --max-warnings 0",
|
|
"format": "prettier --write \"**/*.{ts,tsx,md}\"",
|
|
"storybook": "pnpm run --parallel --recursive storybook:dev",
|
|
"storybook:build": "storybook build",
|
|
"migrations": "cd apps/builder && pnpm migrations --dev",
|
|
"build-figma-tokens": "cd packages/design-system && pnpm build-figma-tokens",
|
|
"prepare": "git config core.hooksPath .git/hooks/ && simple-git-hooks",
|
|
"local:version-snapshot": "pnpm -r exec pnpm version prepatch --preid $(cat /dev/urandom | LC_ALL=C tr -dc 'a-z' | fold -w 8 | head -n 1)",
|
|
"local:publish-snapshot": "pnpm -r publish --access public --no-git-checks --registry http://localhost:4873",
|
|
"local:dangerously-undo-version-snapshot": "git restore --source=$(git branch --show-current) '**/*/package.json'",
|
|
"local:release": "pnpm build && pnpm dts && pnpm local:version-snapshot && pnpm local:publish-snapshot && pnpm local:dangerously-undo-version-snapshot && echo \"now execute\npnpm up -r -L '@webstudio-is/*' --registry http://localhost:4873\""
|
|
},
|
|
"simple-git-hooks": {
|
|
"pre-commit": "./node_modules/.bin/nano-staged"
|
|
},
|
|
"devDependencies": {
|
|
"@storybook/react-vite": "^7.4.0",
|
|
"@types/css-tree": "^2.3.1",
|
|
"@webstudio-is/eslint-config-custom": "workspace:^",
|
|
"esbuild": "^0.19.2",
|
|
"nano-staged": "^0.8.0",
|
|
"prettier": "3.0.3",
|
|
"simple-git-hooks": "^2.9.0",
|
|
"storybook": "^7.4.0",
|
|
"tsx": "^3.12.8",
|
|
"turbo": "^1.8.6",
|
|
"typescript": "5.2.2"
|
|
},
|
|
"engines": {
|
|
"node": "18",
|
|
"pnpm": "8.4",
|
|
"yarn": "This project is configured to use pnpm"
|
|
},
|
|
"nano-staged": {
|
|
"*.{ts,tsx,js,json,css,md}": "prettier --write"
|
|
},
|
|
"prettier": {
|
|
"trailingComma": "es5",
|
|
"overrides": [
|
|
{
|
|
"files": [
|
|
"*.ts",
|
|
"*.tsx"
|
|
],
|
|
"options": {
|
|
"parser": "babel-ts"
|
|
}
|
|
}
|
|
]
|
|
},
|
|
"pnpm": {
|
|
"patchedDependencies": {
|
|
"@stitches/react@1.3.1-1": "patches/@stitches__react@1.3.1-1.patch",
|
|
"css-tree@2.3.1": "patches/css-tree@2.3.1.patch",
|
|
"@types/css-tree@2.3.1": "patches/@types__css-tree@2.3.1.patch",
|
|
"@radix-ui/react-focus-scope@1.0.3": "patches/@radix-ui__react-focus-scope@1.0.3.patch"
|
|
}
|
|
}
|
|
}
|