Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(ui): Adjust platform lists for mobile #28766

Merged
merged 1 commit into from
Sep 22, 2021
Merged

Conversation

k-fish
Copy link
Member

@k-fish k-fish commented Sep 22, 2021

Summary

The platform lists in platform categories should at least have a comprehensive list of platforms including outdated platforms due to doc changes, so that older projects can be matched against.

Long term we should probably export PLATFORM_TO_ICONS from our platformicons library and run tests to ensure platforms in the platform categories files always match against our platform badge list at least.

The mobile category is used in only one case outside of performance that I can see after a quick look, which is the isMobilePlatform check in events titles.

The platform lists in platform categories should at least have a comprehensive list of platforms including outdated platforms due to doc changes, so that older projects can be matched against. Long term we should probably export PLATFORM_TO_ICONs from our platformicons library and run tests to ensure platforms in the platform categories files always match against our platform badge list at least.
@k-fish k-fish requested review from a team, taylangocmen and bruno-garcia September 22, 2021 17:55
Copy link
Contributor

@taylangocmen taylangocmen left a comment

Choose a reason for hiding this comment

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

There are 2 other places I can see, in traces they (mobile platforms) get included in the frontend platforms, and also in project platform picker but I think both are fine to add these on.

@k-fish k-fish merged commit 7f1ad4f into master Sep 22, 2021
@k-fish k-fish deleted the fix/ui-platform-android branch September 22, 2021 20:33
@github-actions github-actions bot locked and limited conversation to collaborators Oct 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants