Skip to content

Conversation

gap-editor
Copy link
Contributor

 I have modified the libs/remix-ui/editor/src/lib/remix-ui-editor.tsx file.

I've added:

  1. A state variable showPasteWarning that checks localStorage when the component mounts to see if the user has previously chosen to hide the warning.

  2. A checkbox labeled "Do not show this warning again" within the "New code pasted" modal.

  3. Logic within the modal's close functions (okFn and cancelFn) to save the preference to localStorage if the checkbox is checked when the modal is dismissed.

Now, when you paste more than 10 lines of code, the warning modal will appear with the new checkbox. If you check it and close the modal (either via "Close" or "Ask RemixAI"), the warning should not appear again in future sessions.

Copy link

netlify bot commented Apr 8, 2025

👷 Deploy request for remixproject pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 0c2cfd7

@Aniket-Engg Aniket-Engg requested review from STetsing and removed request for STetsing April 23, 2025 09:39
@Aniket-Engg
Copy link
Collaborator

@gap-editor build is failing

@gap-editor
Copy link
Contributor Author

@gap-editor build is failing

will work on it on this week

@Aniket-Engg Aniket-Engg requested a review from STetsing April 23, 2025 13:27
@gap-editor
Copy link
Contributor Author

I am going to finish my work this week. Sorry for delay

@gap-editor
Copy link
Contributor Author

Hey! The failing tests don’t seem related to this PR — the errors are coming from GitHub auth steps and build memory issues. This PR doesn’t touch anything related to GitHub integration or build config.

@STetsing
Copy link
Collaborator

STetsing commented May 7, 2025

@gap-editor pls check the lint

668:1   error  Expected indentation of 14 spaces but found 15    indent
  689:41  error  Multiple spaces found before '// Pass true f...'  no-multi-spaces
  721:1   error  Expected indentation of 16 spaces but found 17    indent
  722:1   error  Expected indentation of 18 spaces but found 19    indent
  723:1   error  Expected indentation of 20 spaces but found 21    indent
  724:1   error  Expected indentation of 20 spaces but found 21    indent
  725:1   error  Expected indentation of 20 spaces but found 21    indent
  726:1   error  Expected indentation of 20 spaces but found 21    indent
  727:1   error  Expected indentation of 18 spaces but found 19    indent
  728:1   error  Expected indentation of 18 spaces but found 19    indent
  729:1   error  Expected indentation of 16 spaces but found 17    indent

@Aniket-Engg Aniket-Engg merged commit a76b406 into remix-project-org:master May 19, 2025
24 of 27 checks passed
@STetsing STetsing mentioned this pull request May 21, 2025
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.

3 participants