We don't need update
I'm confused about not needing update. I might be missing some extra context about a new save feature that's coming. Is there a link to some discussion?
/new
flow@ramonjd
I agree, we shouldn't just remove it.
Until the site is launched, we should either:
I threw up https://github.com/Automattic/wp-calypso/pull/42608 without much regard to the consequences.
I'm closing that PR until we come up with a way to address the UX deficit .
Thanks for the input!
Sorry, I should have added more context to issue.
The action saves the doc. Update feels incorrect. We should relabel to "Save".
Rename it to "Save" (I know previous attempt failed)
What failed?
Rename it to "Save" (I know previous attempt failed)
(I think) React had a major hissy fit when it couldn't reconcile what it considered to be the right DOM node tree.
We messed around with a text node, and when it couldn't find it, it threw its toys in the corner and went home, which is a metaphor for throwing an exception and borking the whole site.
Placing on hold until we work out what the best solution would be here.
We can't mess with the text area (See: #42508).
Adding a button is possible (and therefore hiding the existing), we do this with Launch, but there's a move to relocate that to FSE and it seems like an overkill.
Most helpful comment
I threw up https://github.com/Automattic/wp-calypso/pull/42608 without much regard to the consequences.
I'm closing that PR until we come up with a way to address the UX deficit .
Thanks for the input!