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

Revert "Revert "Merge pull request #2344 from filipesilva/react-18"" #2353

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

filipesilva
Copy link
Collaborator

Reverts #2351

@vercel
Copy link

vercel bot commented Sep 7, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
athens ✅ Ready (Inspect) Visit Preview Sep 9, 2022 at 3:58PM (UTC)

@filipesilva
Copy link
Collaborator Author

Problems we found after the react 18 update:

  • typing [[ creates [][]
  • dev auto reload doesn't work
  • some toasts not working

@filipesilva
Copy link
Collaborator Author

So more problems and details:

  • typing a [[ will work, it's just the very first bracket that doesn't get the cursor in the right place (inside)
  • typing a (((((( will add the closing parens, but if you leave that bullet you will something like a )))), and a warning on the console

@filipesilva
Copy link
Collaborator Author

filipesilva commented Sep 9, 2022

Tracked the [][] problem to athens.views.blocks.textarea-keydown, specifically with how handle-pair-char ends up calling (set-cursor-position target new-idx). For some reason it doesn't seem to do anything now if the pair chair is the first char.

@filipesilva
Copy link
Collaborator Author

The )))) problem is weirder... it looks like the bullet has the right content, the textbox too, but the bullet when rendered without editing doesn't?

@filipesilva filipesilva marked this pull request as draft September 9, 2022 19:48
@filipesilva
Copy link
Collaborator Author

Also need to check if the 200ms txs on team graph are related, I saw them around this time but now they are gone.

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.

1 participant