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

Console gets stuck when deleting a tab #1

Closed
eismail-aiopsgroup opened this issue Feb 3, 2023 · 4 comments
Closed

Console gets stuck when deleting a tab #1

eismail-aiopsgroup opened this issue Feb 3, 2023 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@eismail-aiopsgroup
Copy link

eismail-aiopsgroup commented Feb 3, 2023

Hi @z1haze,

The console gets stuck when a tab is deleted. After deleting the tab, I would expect that the previous or a new tab is automatically opened. But instead, a blank editor is shown and the entire page/tab becomes completely unresponsive:
image

Also, the page cannot be refreshed anymore, it just "loads" forever until the browser offers you to force-close the tab.

Error in Chrome Dev Tools:
image

Opening a new console in a new tab works fine and the previously deleted tab is gone as expected.

Browser: Chrome Version 109.0.5414.120 (Official Build) (64-bit)

@z1haze
Copy link
Owner

z1haze commented Feb 3, 2023

Hi @eismail-aiopsgroup,

Sorry about this. I have not had access to a sandbox to thoroughly test my recent changes. I have instead been relying on others to test things for me. I now have the access I need to test the cartridge, and I have already replicated your issue. I will work on a fix for today. Thank you for bringing this up.

@z1haze z1haze self-assigned this Feb 3, 2023
@z1haze z1haze added the bug Something isn't working label Feb 3, 2023
@z1haze z1haze closed this as completed in 9467fe9 Feb 3, 2023
z1haze added a commit that referenced this issue Feb 3, 2023
@z1haze
Copy link
Owner

z1haze commented Feb 3, 2023

@eismail-aiopsgroup, there were a slew of updates following this commit. I think im done updating at the moment, everything appears to be working on my end. Please let me know if when on version 2.0.6 or later, you are still having these issues

@eismail-aiopsgroup
Copy link
Author

eismail-aiopsgroup commented Feb 6, 2023

Hi @z1haze,

I can confirm the issue with the tabs is fixed now. The confirmation popup is also very useful against accidental clicks. Thanks!

The only problem I noticed in v2.2.0 (https://github.com/z1haze/sfcc_dev_console/blob/master/cartridges/sfcc_dev_console/cartridge/controllers/Console.js #90) is that you are using an arrow function which is not supported by older compatibility versions. After replacing it with a regular function syntax, the logging itself works perfectly though.

@z1haze
Copy link
Owner

z1haze commented Feb 6, 2023

Oops, sorry about that -- I often forget that some sites are still on very old compatibility. I can switch that back to a traditional function to maintain backwards compatibility.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants