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

getSelectedBlocks() should ignore blocks where no content is selected #13822

Merged
merged 3 commits into from
Apr 11, 2023

Conversation

filipsobol
Copy link
Member

@filipsobol filipsobol commented Apr 4, 2023

Suggested merge commit message (convention)

Fix (engine): Selection#getSelectedBlocks() should ignore trailing blocks where no content is selected. The selection of such blocks is not visible to the content author and is usually there unintentionally. Closes #11585.

@filipsobol filipsobol force-pushed the ck/11585-fix-range-selection branch from 22df2ff to f4d1628 Compare April 4, 2023 14:30
@filipsobol filipsobol force-pushed the ck/11585-fix-range-selection branch from f4d1628 to 6d6b080 Compare April 4, 2023 15:31
@Reinmar
Copy link
Member

Reinmar commented Apr 5, 2023

Fix (engine): selection should ignore blocks where no content is selected. Closes to #11585.

Corrected:

Fix (engine): Selection#getSelectedBlocks() should ignore trailing blocks where no content is selected. The selection of such blocks is not visible to the content author and is usually there unintentionally. Closes #11585.

@filipsobol filipsobol force-pushed the ck/11585-fix-range-selection branch from abad202 to 2b36cad Compare April 5, 2023 08:29
@filipsobol filipsobol requested a review from Reinmar April 5, 2023 08:31
@filipsobol filipsobol requested a review from arkflpc April 6, 2023 10:50
@arkflpc arkflpc merged commit 9d9604b into master Apr 11, 2023
@arkflpc arkflpc deleted the ck/11585-fix-range-selection branch April 11, 2023 06:18
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.

[Style dropdown] Block style applied to the block before a block widget
3 participants