Skip to content

Reader: Fix colors in post inline comment section#25456

Merged
kean merged 1 commit into
release/26.8from
fix/inline-comments-colors
Mar 26, 2026
Merged

Reader: Fix colors in post inline comment section#25456
kean merged 1 commit into
release/26.8from
fix/inline-comments-colors

Conversation

@kean
Copy link
Copy Markdown
Contributor

@kean kean commented Mar 26, 2026

Fixes CMM-1987: Reader: Incorrect inline comments colors on dark mode with non-dynamic theme

I identified and fixed a couple of more issues when testing Reader Preferences.

Changes

  • Update inline comments to respect ReaderDisplaySettings
  • Fix some previous workarounds (needed more specific qualifiers)
  • Update links in comments to show underlines so it matches the article

Known issue: comment style doesn't refresh when you change a theme (minor)

Screenshot 2026-03-26 at 11 05 27 AM Screenshot 2026-03-26 at 12 16 52 PM

@kean kean added this to the 26.8 ❄️ milestone Mar 26, 2026
@kean kean added the Reader label Mar 26, 2026
@dangermattic
Copy link
Copy Markdown
Collaborator

1 Warning
⚠️ This PR is assigned to the milestone 26.8 ❄️. The due date for this milestone has already passed.
Please assign it to a milestone with a later deadline or check whether the release for this milestone has already been finished.

Generated by 🚫 Danger

}

/* workaround for CMM-1964 */
.reader-full-post.reader-full-post__story-content {
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This actually wasn't working. I'm not sure how I previously tested it. It required a more specific qualifier (dark scheme).

}
}

.reader-full-post__story-content .wp-block-button {
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I included a fix for button style from trunk too.

@sonarqubecloud
Copy link
Copy Markdown

@kean kean changed the title Fix colors in post inline comment section Reader: Fix colors in post inline comment section Mar 26, 2026
@kean kean requested a review from crazytonyli March 26, 2026 16:35
@wpmobilebot
Copy link
Copy Markdown
Contributor

App Icon📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress
ConfigurationRelease-Alpha
Build Number31792
VersionPR #25456
Bundle IDorg.wordpress.alpha
Commit86d83d3
Installation URL0lotm7tvr0un0
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot
Copy link
Copy Markdown
Contributor

App Icon📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack
ConfigurationRelease-Alpha
Build Number31792
VersionPR #25456
Bundle IDcom.jetpack.alpha
Commit86d83d3
Installation URL6vjkb5hf4nekg
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@kean kean merged commit 26c29af into release/26.8 Mar 26, 2026
29 checks passed
@kean kean deleted the fix/inline-comments-colors branch March 26, 2026 19:21
github-merge-queue Bot pushed a commit that referenced this pull request Mar 26, 2026
* Show more detailed application password login errors (#25454)

* Fix colors in post inline comment section (#25456)

* Revert a localizable string change (#25460)

Introduced in #25454, and caused trouble in the release process:

> Inconsistencies found between 'en' and 'zh-Hant':
> `addSite.selfHosted.loadingSiteInfoFailure` expected placeholders for [String] but found [] instead.

* Update strings for localization

* Update app translations – `Localizable.strings`

* Update WordPress metadata translations

* Update Jetpack metadata translations

* Bump version number

---------

Co-authored-by: Tony Li <tony.li@automattic.com>
Co-authored-by: Alex Grebenyuk <alex.grebenyuk@automattic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants