Skip to content

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

Draft
filipesilva wants to merge 4 commits into
mainfrom
revert-2351-revert-react-18
Draft

Revert "Revert "Merge pull request #2344 from filipesilva/react-18""#2353
filipesilva wants to merge 4 commits into
mainfrom
revert-2351-revert-react-18

Conversation

@filipesilva

Copy link
Copy Markdown
Collaborator

Reverts #2351

@vercel

vercel Bot commented Sep 7, 2022

Copy link
Copy Markdown

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
Copy Markdown
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
Copy Markdown
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

filipesilva commented Sep 9, 2022

Copy link
Copy Markdown
Collaborator Author

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
Copy Markdown
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
Copy Markdown
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