Skip to content

Conversation

@wojtekn
Copy link
Contributor

@wojtekn wojtekn commented Apr 11, 2025

Related issues

  • N/A

Proposed Changes

  • I propose to fix the JS error in the About Studio dialog, which prevents all translations from being applied. Also, I propose to make it a little bit wider to account for new link and longer translations.

Testing Instructions

  • Ensure About Studio dialog loads fine, links do not break into multiple lines and that translations load

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?

@wojtekn wojtekn requested a review from a team April 11, 2025 07:08
@wojtekn wojtekn self-assigned this Apr 11, 2025
Copy link
Contributor

@katinthehatsite katinthehatsite left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes look good and I could see the translations loading correctly for me:

Screenshot 2025-04-11 at 9 20 22 AM

Copy link
Contributor

@gavande1 gavande1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes look good to me! 👍

Copy link
Member

@sejas sejas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch!
I confirm I see the translations and I don't see any error when opening Electron → About Studio.

English Spanish
Screenshot 2025-04-11 at 10 10 52 Screenshot 2025-04-11 at 10 11 18

This is the error I saw in the node console on trunk.

Error executing JavaScript: Error: Script failed to execute, this normally means an error was thrown. Check the renderer console for the error.
    at node:electron/js2c/sandbox_bundle:2:127259
    at IpcRendererInternal.<anonymous> (node:electron/js2c/sandbox_bundle:2:121504)
    at IpcRendererInternal.emit (node:electron/js2c/sandbox_bundle:2:45997)
    at Object.onMessage (node:electron/js2c/sandbox_bundle:2:119481)

@ivan-ottinger
Copy link
Contributor

Thanks for the fix, Wojtek!

@wojtekn wojtekn merged commit 8ace467 into trunk Apr 11, 2025
8 checks passed
@wojtekn wojtekn deleted the fix/about-studio-dialog branch April 11, 2025 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants