Skip to content

Conversation

@kean
Copy link
Contributor

@kean kean commented Jan 22, 2026

Replaces the floating section header view – which can be pretty annoying – with a small button in the navigation bar. You get to this screen from the Notifications.

Before:
https://github.com/user-attachments/assets/1ad4ddaa-e6d7-4a83-88be-aa5b7d765bec

After:
https://github.com/user-attachments/assets/17c6cee1-cc37-468d-9644-8cd0ce6b64b2

@kean kean added the Reader label Jan 22, 2026
@kean kean added this to the 26.7 milestone Jan 22, 2026
@wpmobilebot
Copy link
Contributor

wpmobilebot commented Jan 22, 2026

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 Number30551
VersionPR #25159
Bundle IDcom.jetpack.alpha
Commit93957ca
Installation URL6p996cqpe7om0
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@kean kean force-pushed the task/replace-comment-header-view branch from 6b5a357 to 93957ca Compare January 22, 2026 19:37
@sonarqubecloud
Copy link

@wpmobilebot
Copy link
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 Number30551
VersionPR #25159
Bundle IDorg.wordpress.alpha
Commit93957ca
Installation URL6lkrgol5lv098
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

Copy link
Contributor

@jkmassel jkmassel left a comment

Choose a reason for hiding this comment

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

This fixes a Liquid Glass-related issue too, so it's a big win.

subtitleLabel.textAlignment = .center
subtitleLabel.maximumContentSizeCategory = .extraLarge

let chevronImageView = UIImageView()
Copy link
Contributor

Choose a reason for hiding this comment

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

Something about the image seems off – I'm not sure if it's the text baseline or if it's not quite the right size.

Will it work with dynamic type with a hard-coded point size?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The labels can only grow so much: subtitleLabel.maximumContentSizeCategory = .extraLarge. There isn't space in the navigation bar. Let me check again how well it looks with the maximum size.

@wpmobilebot
Copy link
Contributor

🤖 Build Failure Analysis

This build has failures. Claude has analyzed them - check the build annotations for details.

@kean kean marked this pull request as draft January 22, 2026 21:41
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