Fix three "be the customer" UI issues (padding, plans inset, search spinner) - #23289
Open
adalpari wants to merge 2 commits into
Open
Fix three "be the customer" UI issues (padding, plans inset, search spinner)#23289adalpari wants to merge 2 commits into
adalpari wants to merge 2 commits into
Conversation
- CMM-2375: add internal padding inside the pre-publish social box so its content is not flush against the card edges - CMM-2371: consume system-bar insets on the site-creation plans Scaffold so the status-bar inset is not applied twice on first load - CMM-2382: apply the new filter with the loading state so the first Free Photo Library search shows its spinner Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Collaborator
Generated by 🚫 Danger |
Extract applyFilterFor() so the loading state and loaded state derive the filter from a single source. Fixes the Start action being omitted from the loading-state filter mapping (CMM-2382 latent case) and removes the duplicated mapping that could drift. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Contributor
|
|
Contributor
|
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## trunk #23289 +/- ##
=======================================
Coverage 37.91% 37.92%
=======================================
Files 2352 2352
Lines 128795 128800 +5
Branches 17898 17901 +3
=======================================
+ Hits 48836 48841 +5
Misses 75964 75964
Partials 3995 3995 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
adalpari
marked this pull request as ready for review
September 3, 2026 12:18
Contributor
|
@adalpari Changes look good! Claude flagged a potential problem. |
nbradbury
approved these changes
Sep 3, 2026
nbradbury
left a comment
Contributor
There was a problem hiding this comment.
I'll go ahead and approve this and leave it to you to decide if Claude's finding is worth addressing ![]()
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


TL;DR
This PR is fixing a set of general visual errors: margins, paddings, and a loading spinner.
Description
Three small fixes found during the "Be the Customer" support rotation, all verified on-device (Jetpack, Pixel 7):
PostSocialSharingItemis only used in this sheet).BaseAppCompatActivityalready pads the window for system bars on edge-to-edge (Android 15+), andSiteCreationActivityis not excluded, but the plans fragment's ComposeScaffold+TopAppBarapplied the status-bar inset again — offsetting the whole screen by the status-bar height. The first-load-vs-reopen flip was an inset-consumption race. Fixed by consuming the system-bar insets so the screen relies solely on the activity's padding, like its XML sibling screens.Testing instructions
Pre-publish social box padding (CMM-2375):
Plans screen top margin (CMM-2371):
Before / After


Free Photo Library first-search spinner (CMM-2382):