Skip to content

Remove Platform-specific Android check from app screen header#14086

Open
dylanjeffers wants to merge 1 commit intomainfrom
claude/add-android-logo-NgQD8
Open

Remove Platform-specific Android check from app screen header#14086
dylanjeffers wants to merge 1 commit intomainfrom
claude/add-android-logo-NgQD8

Conversation

@dylanjeffers
Copy link
Copy Markdown
Contributor

Summary

Removed the Android-specific platform check that was preventing the Audius logo from displaying in the app screen header on Android devices.

Key Changes

  • Removed Platform import from react-native (no longer needed)
  • Removed the conditional if (Platform.OS === 'android') return null check that was hiding the header logo on Android
  • The Audius logo header will now display consistently across all platforms

Implementation Details

This change allows the IconAudiusLogoHorizontal component to render on Android devices, matching the behavior on iOS. The logo will now be visible in the app screen header navigation for all users.

https://claude.ai/code/session_01XGhKL4tFJMo6L9T67NykfQ

The header title was returning null on Android, hiding the Audius
logo that renders when no screen title is set. Let the logo render
on both platforms for consistent branding.

https://claude.ai/code/session_01XGhKL4tFJMo6L9T67NykfQ
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 8, 2026

⚠️ No Changeset found

Latest commit: d255014

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

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.

2 participants