wordpress-mobile / WordPress-Android

WordPress for Android
http://android.wordpress.org
GNU General Public License v2.0
2.99k stars 1.33k forks source link

Material 3: My Site Cards #21439

Closed nbradbury closed 1 week ago

nbradbury commented 1 week ago

This PR converts all of the My Site cards to Material 3. I considered converting the entire My Site section to M3 but that would require more changes than I would like in a single PR, so instead I'm splitting the conversion into sections.

To test:

dangermattic commented 1 week ago
2 Warnings
:warning: View files have been modified, but no screenshot or video is included in the pull request. Consider adding some for clarity.
:warning: PR is not assigned to a milestone.

Generated by :no_entry_sign: Danger

wpmobilebot commented 1 week ago
WordPress📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress WordPress
FlavorJalapeno
Build TypeDebug
Versionpr21439-c0fb895
Commitc0fb895f73aaf319e385cce1f6beb9beda4db3f3
Direct Downloadwordpress-prototype-build-pr21439-c0fb895.apk
Note: Google Login is not supported on these builds.
wpmobilebot commented 1 week ago
Jetpack📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack Jetpack
FlavorJalapeno
Build TypeDebug
Versionpr21439-c0fb895
Commitc0fb895f73aaf319e385cce1f6beb9beda4db3f3
Direct Downloadjetpack-prototype-build-pr21439-c0fb895.apk
Note: Google Login is not supported on these builds.
codecov[bot] commented 1 week ago

Codecov Report

Attention: Patch coverage is 0% with 16 lines in your changes missing coverage. Please review.

Project coverage is 39.50%. Comparing base (17f9039) to head (c0fb895). Report is 19 commits behind head on trunk.

Files with missing lines Patch % Lines
.../mysite/cards/dynamiccard/DynamicCardComposable.kt 0.00% 6 Missing :warning:
...ndroid/ui/mysite/cards/blaze/BlazeCampaignsCard.kt 0.00% 3 Missing :warning:
.../android/ui/mysite/cards/nocards/NoCardsMessage.kt 0.00% 3 Missing :warning:
.../cards/dashboard/bloganuary/BloganuaryNudgeCard.kt 0.00% 2 Missing :warning:
...oid/ui/mysite/cards/personalize/PersonalizeCard.kt 0.00% 1 Missing :warning:
...id/ui/mysite/cards/sotw2023/WpSotw2023NudgeCard.kt 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## trunk #21439 +/- ## ======================================= Coverage 39.50% 39.50% ======================================= Files 2121 2120 -1 Lines 99444 99438 -6 Branches 15284 15282 -2 ======================================= Hits 39281 39281 + Misses 56681 56675 -6 Partials 3482 3482 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

sonarcloud[bot] commented 1 week ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

sentry-io[bot] commented 1 week ago

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

Did you find this useful? React with a 👍 or 👎