Skip to content
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

Errors on the frontend console when opening Papaya #2647

Closed
pcdavid opened this issue Nov 28, 2023 · 1 comment · Fixed by #2656
Closed

Errors on the frontend console when opening Papaya #2647

pcdavid opened this issue Nov 28, 2023 · 1 comment · Fixed by #2656

Comments

@pcdavid
Copy link
Member

pcdavid commented Nov 28, 2023

Open a Papaya diagram (React Flow, using the initial data).

In the frontend's console I see a lot of errors like this:

console.js:213 Warning: Removing a style property during rerender (borderBottom) when a conflicting property is set (borderWidth) can lead to styling bugs. To avoid this, don't mix shorthand and non-shorthand properties for the same value; instead, replace the shorthand with separate values.
    at div
    at http://localhost:5173/@fs/home/pcdavid/work/ocp/git/sirius-web/packages/diagrams/frontend/sirius-components-diagrams-reactflow/dist/sirius-components-diagrams-reactflow.es.js:2452:26
    at div
    at NodeWrapper (http://localhost:5173/node_modules/.vite/deps/reactflow.js?v=8cae8206:2816:26)
    at div
    at NodeRenderer (http://localhost:5173/node_modules/.vite/deps/reactflow.js?v=8cae8206:3032:115)
    at div
    at Viewport (http://localhost:5173/node_modules/.vite/deps/reactflow.js?v=8cae8206:3452:21)
    at div
    at http://localhost:5173/node_modules/.vite/deps/reactflow.js?v=8cae8206:2344:39
    at div
    at ZoomPane (http://localhost:5173/node_modules/.vite/deps/reactflow.js?v=8cae8206:1972:19)
    at FlowRenderer (http://localhost:5173/node_modules/.vite/deps/reactflow.js?v=8cae8206:2979:23)
    at GraphView (http://localhost:5173/node_modules/.vite/deps/reactflow.js?v=8cae8206:3560:20)
    at Wrapper (http://localhost:5173/node_modules/.vite/deps/reactflow.js?v=8cae8206:3835:18)
@pcdavid
Copy link
Member Author

pcdavid commented Nov 28, 2023

According to a git bisect, this was introduced by d038e39.
Edit: maybe not actually.

gcoutable added a commit that referenced this issue Nov 28, 2023
Bug: #2647
Signed-off-by: Guillaume Coutable <guillaume.coutable@obeo.fr>
@gcoutable gcoutable linked a pull request Nov 28, 2023 that will close this issue
39 tasks
@gcoutable gcoutable self-assigned this Nov 28, 2023
gcoutable added a commit that referenced this issue Nov 28, 2023
Bug: #2647
Signed-off-by: Guillaume Coutable <guillaume.coutable@obeo.fr>
gcoutable added a commit that referenced this issue Nov 28, 2023
Bug: #2647
Signed-off-by: Guillaume Coutable <guillaume.coutable@obeo.fr>
sbegaudeau pushed a commit that referenced this issue Nov 28, 2023
Bug: #2647
Signed-off-by: Guillaume Coutable <guillaume.coutable@obeo.fr>
sbegaudeau pushed a commit that referenced this issue Nov 28, 2023
Bug: #2647
Signed-off-by: Guillaume Coutable <guillaume.coutable@obeo.fr>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants