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.

Sign up for GitHub

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

Jump to bottom

Bug: Breaks on chromeOS #17

Closed
Unexian opened this issue Nov 15, 2022 · 8 comments
Closed

Bug: Breaks on chromeOS #17

Unexian opened this issue Nov 15, 2022 · 8 comments
Labels
bug Something isn't working

Comments

@Unexian
Copy link

Unexian commented Nov 15, 2022 •
edited
Loading

Describe the current behavior

When I try to place an operation, like the viginere cipher operation, the ui disappears and I cannot use the website

Describe the expected behavior

The operation gets placed and the ui remains on the screen

Steps to reproduce the bug

  1. Get a chromebook or similar
  2. Navigate to ciphereditor.com.
  3. Try to place an operation «EDIT: also happens when exiting a program operation»

Version, Environment, Platform

  • ciphereditor version: I cannot find where it says the version
  • Browser and browser version (if applicable): Google Chrome 107.0.5304.110 x64
  • Operating system: chromeOS
@Unexian Unexian added the bug Something isn't working label Nov 15, 2022
@ffraenz
Copy link
Member

ffraenz commented Nov 15, 2022

Oh, that's interesting! Are you able to post the content of the dev tools console (stack trace or screenshot of it)? I have no Chrome OS at hand, right now.

@Unexian
Copy link
Author

Unexian commented Nov 15, 2022 •
edited by ffraenz
Loading

when adding a viginere cipher operation:

index.bc3e647f.js:57 Error: Minified React error #185; visit https://reactjs.org/docs/error-decoder.html?invariant=185 for the full message or use the non-minified dev environment for full errors and additional helpful warnings.
    at Ct (index.bc3e647f.js:57:33996)
    at wv (index.bc3e647f.js:55:22379)
    at Object.callback (index.bc3e647f.js:55:22242)
    at index.bc3e647f.js:72:2081
    at Id (index.bc3e647f.js:57:37135)
    at Object.notify (index.bc3e647f.js:72:2056)
    at Object.i [as notifyNestedSubs] (index.bc3e647f.js:72:2469)
    at a (index.bc3e647f.js:72:2510)
    at h (index.bc3e647f.js:2:86481)
    at index.bc3e647f.js:72:67366
Xu @ index.bc3e647f.js:57
Pv.n.callback @ index.bc3e647f.js:57
tp @ index.bc3e647f.js:55
gp @ index.bc3e647f.js:57
Yv @ index.bc3e647f.js:57
Zb @ index.bc3e647f.js:57
Gb @ index.bc3e647f.js:57
Bn @ index.bc3e647f.js:57
xp @ index.bc3e647f.js:57
Vn @ index.bc3e647f.js:55
(anonymous) @ index.bc3e647f.js:57
index.bc3e647f.js:57 Uncaught Error: Minified React error #185; visit https://reactjs.org/docs/error-decoder.html?invariant=185 for the full message or use the non-minified dev environment for full errors and additional helpful warnings.
    at Ct (index.bc3e647f.js:57:33996)
    at wv (index.bc3e647f.js:55:22379)
    at Object.callback (index.bc3e647f.js:55:22242)
    at index.bc3e647f.js:72:2081
    at Id (index.bc3e647f.js:57:37135)
    at Object.notify (index.bc3e647f.js:72:2056)
    at Object.i [as notifyNestedSubs] (index.bc3e647f.js:72:2469)
    at a (index.bc3e647f.js:72:2510)
    at h (index.bc3e647f.js:2:86481)
    at index.bc3e647f.js:72:67366

@Unexian
Copy link
Author

Unexian commented Nov 15, 2022 •
edited by ffraenz
Loading

before i placed the operation:

content-script.js:1 Sending message to background page sw-shouldActivate
content-script.js:1 Hiding page content
content-script.js:1 Successfully hid page content
content-script.js:1 Received message from background page: "sw-rescan"
content-script.js:1 Ignoring first request for re-scan
content-script.js:1 Received message from background page: "sw-rescan"
content-script.js:1 Re-scanning page content
content-script.js:1 Running scrape dom 1
content-script.js:1 Sending message to background page {pageContent: '<html lang="en-us" class="root root--script-enable…ormal;"></p></next-route-announcer></body></html>', pageLocation: 'https://ciphereditor.com/?utm_campaign=cryptii&amp;utm_source=referrer&amp;utm_medium=textlink', pageTitle: 'Your playground for cryptography, coding & data - ciphereditor', rescan: false}
content-script.js:1 Sending message to background page sw-shouldActivate
content-script.js:1 Running scrape dom 1
content-script.js:1 Sending message to background page {pageContent: '<html lang="en-us" class="root root--script-enable…n0%3d" style="position: absolute;"></body></html>', pageLocation: 'https://ciphereditor.com/?utm_campaign=cryptii&amp;utm_source=referrer&amp;utm_medium=textlink', pageTitle: 'Your playground for cryptography, coding & data - ciphereditor', rescan: Array(1)}
content-script.js:1 Received message from background page: {
 "actionCode": [
  3
 ],
 "contentModsToApply": [
  "477"
 ],
 "debug": false,
 "metadataVersion": 3
}
content-script.js:1 Received message to modify dom
content-script.js:1 Displaying page again
content-script.js:1 Received message from background page: {
 "actionCode": [
  3
 ],
 "contentModsToApply": [
  "477"
 ],
 "debug": false,
 "metadataVersion": 3
}
content-script.js:1 Received message to modify dom
content-script.js:1 Displaying page again
content-script.js:1 Received message from background page: {
 "actionCode": [
  3
 ],
 "contentModsToApply": [
  "477"
 ],
 "debug": false,
 "metadataVersion": 3
}
content-script.js:1 Received message to modify dom
content-script.js:1 Displaying page again
content-script.js:1 Received message from background page: {
 "actionCode": [
  3
 ],
 "contentModsToApply": [
  "477"
 ],
 "debug": false,
 "metadataVersion": 3
}
content-script.js:1 Received message to modify dom
content-script.js:1 Displaying page again
content-script.js:1 Hiding page content
content-script.js:1 Successfully hid page content
content-script.js:1 Running scrape dom 1
content-script.js:1 Sending message to background page {pageContent: '<html lang="en" class="root root--script-disabled …block; left: 0px; top: 0px;"></div></body></html>', pageLocation: 'https://app.ciphereditor.com/', pageTitle: 'ciphereditor', rescan: false}
content-script.js:1 Received message from background page: {
 "actionCode": [
  3
 ],
 "contentModsToApply": [
  "477"
 ],
 "debug": false,
 "metadataVersion": 3
}
content-script.js:1 Received message to modify dom
content-script.js:1 Displaying page again
content-script.js:1 Received message from background page: {
 "actionCode": [
  3
 ],
 "contentModsToApply": [
  "477"
 ],
 "debug": false,
 "metadataVersion": 3
}
content-script.js:1 Received message to modify dom
content-script.js:1 Displaying page again

@ffraenz
Copy link
Member

ffraenz commented Nov 15, 2022

I just deployed a new version where sourcemaps are present and where Sentry, an error reporting tool, is configured. Would you revisit the app? I should receive a more detailed error report that hopefully gives me a hint about where in the code the issue lies. I'll install chrome OS in the meantime.

@Unexian
Copy link
Author

Unexian commented Nov 16, 2022

it works

@Unexian
Copy link
Author

Unexian commented Nov 16, 2022

ok yea it works

@Unexian Unexian closed this as completed Nov 16, 2022
@ffraenz
Copy link
Member

ffraenz commented Nov 16, 2022

Okay, well. The update is deploying as we speak, I'm not sure why it works already for you :)

I received a similar issue causing the same React error "Maximum update depth exceeded" and found out that it has something to do with how nodes sizes and positions are measured on the canvas. I refactored this code and switched to a different strategy (using ResizeObservers).

Would you hit refresh (once again) and check if it still works after the deployment? It just went through.

@Unexian
Copy link
Author

Unexian commented Nov 16, 2022

I am not at school atm

@Unexian Unexian mentioned this issue Nov 17, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ffraenz @Unexian