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

Add a check for Experimental Feature DefaultFormatInSpan #1469

Merged
merged 3 commits into from
Dec 16, 2022

Conversation

BryanValverdeU
Copy link
Contributor

In #1380, we introduced adding Format in Span instead of DIV.

We added an Experimental Feature but we did not use it to modify the behavior.
Since we added new enums to the repo we cannot fully revert as it would be a breaking change.

So the fix I propose is to start using the Experimental Feature DefaultFormatInSpan to decide if we want the new behavior to add format to the block div or to the span that is nested in the div.

@BryanValverdeU BryanValverdeU changed the title Partially revert #1380 Add a check for Experimental Feature DefaultFormatInSpan Dec 16, 2022
@BryanValverdeU BryanValverdeU merged commit ea4ea45 into master Dec 16, 2022
juliaroldi added a commit that referenced this pull request Dec 20, 2022
@BryanValverdeU BryanValverdeU mentioned this pull request Dec 27, 2022
BryanValverdeU added a commit that referenced this pull request Dec 30, 2022
* Revert revert

* Remove position change

* Fix Tests

Co-authored-by: Julia Roldi <[email protected]>
BryanValverdeU added a commit that referenced this pull request Jan 4, 2023
* Revert revert

* Remove position change

* Fix Tests

Co-authored-by: Julia Roldi <[email protected]>
@JiuqingSong JiuqingSong deleted the u/bvalverde/useExpFeature branch September 8, 2023 22:33
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.

2 participants