Block Comments: Graduate Block Level Commenting from Experiments#72181
Block Comments: Graduate Block Level Commenting from Experiments#72181roseg43 wants to merge 5 commits intoWordPress:trunkfrom
Conversation
… JavaScript flag. Removes the experimental flag as part of graduating the Block Comments feature out of experimental and into compat/6.9.
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
…s view. Additionally, remove experimental flags and conditionals from Block Comments test code.
fabiankaegy
left a comment
There was a problem hiding this comment.
The code here looks good to me 👍
I've tested the feature and it works 👍
I don't have enough insight into the overall feature direction at this point. To me, it still feels a bit unpolished (especially in the UI flows) but again I haven't been involved at all in the 6.9 scope planning etc :)
Would love a final confirmation from someone like @Mamaduka :) before we merge 👍
|
I think a couple more enhancements still need to go in before we graduate this feature from experimental. |
|
Hi, @roseg43 Would you be able to rebase this branch and resolve merge conflicts? We should also create a core sync PR and ticket. I'm happy to handle that. |
|
@Mamaduka i believe @roseg43 is offline for the next couple of days, so if you (or alternatively @adamsilverstein @poojabhimani12) are able to resolve conflicts to get this ready for merge that's probably best. |
|
@jeffpaul, I'll handle it. Though I might need to create a new PR, it depends on whether I can push to the forked branch. |
|
Here's the new PR - #72344. |
|
@Mamaduka we can likely close this PR now as yours has been merged, yeah? |
Graduates the Block Commenting feature from experimental to compat for 6.9.
What?
Closes #71826
This PR moves all Block Comments-related code located in
lib/experimentaltolib/compat/wordpress-6.9. Additionally, it removes all experimental flags for conditionally enabling Block Comments functionality.Why?
From #71826 :
How?
This PR executes the steps outlined in this comment:
Testing Instructions
Confirm all commenting functionality is available
Confirm the toggle option to enable the feature experiment has been removed