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

Fix #2857 #2858

Merged
merged 1 commit into from
Nov 4, 2024
Merged

Fix #2857 #2858

merged 1 commit into from
Nov 4, 2024

Conversation

JiuqingSong
Copy link
Collaborator

#2857
Bold style is missing when ENTER on list item

This is because when create new list item, instead of inheriting list item format from previous list item, we actually inherit format from current selection marker, that causes bold format is incorrectly set into list item format holder, which causes when recreate the new list item, it thoughts there is already bold format applied in container level, so bold format is skipped.

Fix: Inherit format from previous list item instead.

@JiuqingSong JiuqingSong merged commit c2051c7 into master Nov 4, 2024
7 checks passed
@JiuqingSong JiuqingSong deleted the u/jisong/2857 branch November 4, 2024 20:58
BryanValverdeU added a commit that referenced this pull request Nov 15, 2024
* fix test

* test

* Merge text node and segments (#2846)

* Merge text segments

* Fix test

* merge node

* fix build and test

* add test

* Add test

* fix test

* Remove tablePreProcessor (#2849)

* Add change data and apiName to ContentChangedEvent when handle keyboard input (#2854)

* Fix #2857 (#2858)

* Revert Shortcut command for Clear Format on mac (#2834)

* revert Shortcut command for Clear Format on mac

* fix ctrl and meta can press in the same time

* update comment

* Allow browser's default paste behavior when pasting from Office Android (#2863)

* Implement shouldPreventDefaultPaste function

* fix build

* fix build

* list-bugs

* fix image floating

* Customize the behavior of merging format values (#2865)

* define types

* add callbacks

* use param instead

* fix test

---------

Co-authored-by: Jiuqing Song <jisong@microsoft.com>

* updateVersion.json

* Fix merge segment issue (#2871)

---------

Co-authored-by: Julia Roldi (from Dev Box) <juliaroldi@microsoft.com>
Co-authored-by: Julia Roldi <87443959+juliaroldi@users.noreply.github.com>
Co-authored-by: Jiuqing Song <jisong@microsoft.com>
Co-authored-by: 庄黛淳华 <miku1958@users.noreply.github.com>
Co-authored-by: Rain-Zheng <67583056+Rain-Zheng@users.noreply.github.com>
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