Skip to content

fix(android): removing inline styles#672

Open
kacperzolkiewski wants to merge 1 commit into
mainfrom
@kacperzolkiewski/fix-applying-bold
Open

fix(android): removing inline styles#672
kacperzolkiewski wants to merge 1 commit into
mainfrom
@kacperzolkiewski/fix-applying-bold

Conversation

@kacperzolkiewski

@kacperzolkiewski kacperzolkiewski commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

Summary

Fixes: #671

This PR fixes removing inline styles within a range while preserving the styles before and after the selected range.

Test Plan

Run reproduction steps from issue: #671
The issue should be resolved.

Screenshots / Videos

Before:

Screen.Recording.2026-07-03.at.10.33.54.mov

After:

Screen.Recording.2026-07-03.at.10.43.23.mov

Compatibility

OS Implemented
iOS
Android
Web

Checklist

  • E2E tests are passing
  • Required E2E tests have been added (if applicable)

@kacperzolkiewski kacperzolkiewski marked this pull request as ready for review July 3, 2026 09:12

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR addresses Android span handling when removing conflicting inline styles, aiming to ensure inline styles are removed only within the requested [start, end] range while preserving the same style applied before and after that range (fix for #671).

Changes:

  • Update Android inline-style removal to split affected spans and re-apply the preserved left/right portions outside the removal range.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

[Bug] Android: applying a bold heading strips bold from the previous line

2 participants