Skip to content

Conversation

@akabiru
Copy link
Member

@akabiru akabiru commented Dec 4, 2025

The hash function is not consistently portable between JS/Ruby due to different "integer overflow handling (!?)". JS based coloring establishes consistency whilst we have both PVC and the legacy (angular) user avatar component rendering avatar fallbacks.

The hash function is not consistently portable between JS/Ruby due to different "integer overflow handling (!?)".
JS based coloring establishes consistency while we have both primer legacy (angular) component rendering avatar fallback
@changeset-bot
Copy link

changeset-bot bot commented Dec 4, 2025

⚠️ No Changeset found

Latest commit: dd0f335

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

@akabiru akabiru changed the title Mimic OpenProjec Core JS based color hash function for consistency bug/69230 Mimic OpenProjec Core JS based color hash function for consistency Dec 4, 2025
@github-actions
Copy link

github-actions bot commented Dec 4, 2025

⚠️ Visual or ARIA snapshot differences found

Our visual and ARIA snapshot tests found UI differences. Please review the differences by viewing the files changed tab to ensure that the changes were intentional.

Review differences

@akabiru
Copy link
Member Author

akabiru commented Dec 4, 2025

Superseded by #387

@akabiru akabiru closed this Dec 4, 2025
@akabiru akabiru deleted the bug/69230-fix-avatars-with-initials-Mimic-OP-Core branch December 4, 2025 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants