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

Unable to type curly bracket inside of R Sweave code chunk #14667

Closed
4 of 6 tasks
jdblischak opened this issue May 7, 2024 · 8 comments
Closed
4 of 6 tasks

Unable to type curly bracket inside of R Sweave code chunk #14667

jdblischak opened this issue May 7, 2024 · 8 comments

Comments

@jdblischak
Copy link
Contributor

jdblischak commented May 7, 2024

Verify fixed in:

  • Patch: 2024.04.2
  • Release: 2024.07.0

System details

RStudio Edition : Desktop
RStudio Version : 2024.04.0+735
OS Version      : Windows 11 x64 (build 22631)
R Version       : 4.3.3 (2024-02-29 ucrt)

Steps to reproduce the problem

  1. Open a new R Sweave file (File -> New File -> R Sweave)
  2. Add a code chunk (Ctrl-Alt-I or push the button in the editor UI)
  3. Put the cursor inside of the code chunk. Try to type a curly bracket {

Describe the problem in detail

When I follow the steps above, the curly bracket does not appear. Instead I get the following Autosave error:

image

The above is the quickest way to generate the problem. However, attempting to add the curly bracket can cause chaos at any point. For example, if you have code in the code chunk and then add the curly bracket, the code is erased.

Please see this Posit Community post for the description from the original reporter as well as my reproduction of the bug.

Describe the behavior you expected

I expected a curly bracket to appear inside the code chunk.

  • I have read the guide for submitting good bug reports.
  • I have installed the latest version of RStudio, and confirmed that the issue still persists.
  • If I am reporting an RStudio crash, I have included a diagnostics report.
  • I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
@jdblischak jdblischak added bug new New incoming issues, ready for initial review. labels May 7, 2024
@ronblum ronblum added regression and removed new New incoming issues, ready for initial review. labels May 7, 2024
@ronblum
Copy link
Contributor

ronblum commented May 8, 2024

@jdblischak Thank you for raising this! We're tracking this bug in issue #14646, so I'll close this one as a duplicate.

@ronblum
Copy link
Contributor

ronblum commented May 8, 2024

Actually I'll keep this open because of the order of events and use both issues for verification.

@ronblum ronblum reopened this May 8, 2024
@ronblum ronblum removed the duplicate label May 8, 2024
@jdblischak
Copy link
Contributor Author

so I'll close this one as a duplicate.

Wow! This repo gets a lot of Issues per day. I skimmed the first couple open and closed Issues and PRs before opening mine, but clearly that was insufficient for such a popular repository. Thanks for the quick reply!

@ronblum
Copy link
Contributor

ronblum commented May 9, 2024

True! It's especially popular right now, since we just had a new release last week, and we're already working on the next release. I'm glad for it, because it means that all y'all care enough to let us know about bugs that you find and improvements that you like. 🙂 Plus the development team never seems sleep, although this QA engineer is often sleeping on the job.

@ronblum
Copy link
Contributor

ronblum commented May 15, 2024

Verified fixed in RStudio Desktop 2024.07.0-daily+133 on Windows 11. If you'd like to try out the fixed, you can download a daily build at https://dailies.rstudio.com. Please note that these are for testing purposes only, not for production use.

@ronblum
Copy link
Contributor

ronblum commented May 29, 2024

Update: We have a fix for this in an upcoming patch. You can get a development version of patch 2024.04.2 at https://dailies.rstudio.com/rstudio/chocolate-cosmos/. The usual caveats apply--this is for testing purposes only, not production.

@ronblum
Copy link
Contributor

ronblum commented May 29, 2024

See automated tests: https://github.com/rstudio/rstudio-ide-automation/pull/1949

Verified in RStudio Desktop 2024.04.2+761 on Windows 11.

@ronblum ronblum self-assigned this May 29, 2024
@ronblum
Copy link
Contributor

ronblum commented May 29, 2024

Verified in RStudio Desktop 2024.07.0-daily+174 on Windows 11 using the same tests.

@ronblum ronblum closed this as completed May 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants