Skip to content

Conversation

akib22
Copy link
Contributor

@akib22 akib22 commented Oct 9, 2025

Description

This pull request addresses a bug where the UI language changes immediately upon selecting a language from the modal, even if the user does not click the "Confirm" button. Previously, closing or canceling the modal would still apply the selected language, which was unexpected behavior.

With this fix, the UI only applies the new language when the user explicitly confirms their choice. If the modal is closed or canceled, the language remains unchanged.

Changes Made

  • Updated language change modal logic to apply language selection only after confirmation
  • Prevented UI updates when the modal is closed or canceled without confirmation
  • Tested modal behavior for all supported languages

Video

fix-drawdb-language-selection.mov

Related Issue

Closes #632

Checklist

  • Language changes only apply after confirmation
  • Modal cancel/close does not change UI language
  • No regressions or unexpected side effects

#hacktoberfest

akib22 and others added 3 commits October 9, 2025 02:31
- ordered the translation keys same as en.js file to check the difference easily
- improve word selection and sentence structure

Issue: drawdb-io#627
Copy link

vercel bot commented Oct 9, 2025

@akib22 is attempting to deploy a commit to the dottle's projects Team on Vercel.

A member of the Team first needs to authorize it.

@akib22 akib22 marked this pull request as draft October 9, 2025 09:17
@akib22 akib22 marked this pull request as ready for review October 9, 2025 17:13
@akib22
Copy link
Contributor Author

akib22 commented Oct 11, 2025

Hi @1ilit,

I hope you're doing well. Could you please review this PR when you have a moment? I apologize for mistakenly creating this PR on the fork's main branch. Because of this, whenever I try to work on other issues in different branches, I have to remove these changes by rebasing, which is a bit inconvenient.

It would be a great help if you could merge this PR so I can continue working without any hassle. I really appreciate your time and support.

Thank you so much!

Copy link

vercel bot commented Oct 11, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
drawdb Ready Ready Preview Comment Oct 11, 2025 10:47pm

Copy link
Member

@1ilit 1ilit left a comment

Choose a reason for hiding this comment

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

thanks, please just make sure the changes concern the issue youre fixing. it's just imports and minor stuff now, but for future reference

@1ilit 1ilit merged commit a48e6ef into drawdb-io:main Oct 11, 2025
3 checks passed
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.

[BUG] Language Change Applied Without Confirmation in Modal

2 participants