-
Notifications
You must be signed in to change notification settings - Fork 14
Open
Labels
P1Medium PriorityMedium PriorityenhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomersperformanceperformance related issuesperformance related issuessomedayThings we're not ready to tackle yet, but will somedayThings we're not ready to tackle yet, but will someday
Milestone
Description
- Right now we use a mixture of icons from five icon sets. This isn't good from a memory usage or a visual consistency standpoint. Designers have been using the AntDesign icons so we should probably follow suit.
❯ rg "import.*vector-icons"
components/AvalancheForecastZoneMap.tsx
20:import {FontAwesome5} from '@expo/vector-icons';
components/TelemetryStationMap.tsx
4:import {FontAwesome5} from '@expo/vector-icons';
components/icons/nac-icons.tsx
1:import {createIconSet} from '@expo/vector-icons';
components/content/Carousel.tsx
5:import {AntDesign, FontAwesome5} from '@expo/vector-icons';
components/content/Card.tsx
6:import {FontAwesome} from '@expo/vector-icons';
components/DangerScale.tsx
3:import {MaterialIcons} from '@expo/vector-icons';
components/Observations.tsx
8:import {FontAwesome, MaterialCommunityIcons} from '@expo/vector-icons';
components/Observation.tsx
5:import {FontAwesome5, MaterialCommunityIcons, Fontisto} from '@expo/vector-icons';
App.tsx
7:import {AntDesign} from '@expo/vector-icons';
- We should preload icons so they're ready to go when we need them - https://docs.expo.dev/archive/classic-updates/preloading-and-caching-assets/#pre-loading-and-caching-assets
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
P1Medium PriorityMedium PriorityenhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomersperformanceperformance related issuesperformance related issuessomedayThings we're not ready to tackle yet, but will somedayThings we're not ready to tackle yet, but will someday