AAO-React-Native icon indicating copy to clipboard operation
AAO-React-Native copied to clipboard

Followups for the rn-vector-icons@6 upgrade

Open hawkrives opened this issue 6 years ago • 6 comments

Regarding #3459:

These are the matches for ios- prefixed icons in our repo:

  • [x] modules/navigation-buttons/open-settings.js <Icon name="ios-settings" style={leftButtonStyles.icon} />

  • [x] modules/lists/disclosure-arrow.js – no action needed; Android short-circuits before rendering the icon

  • [x] source/views/home/edit/row.ios.js – no action needed; Android will never run this file

  • [ ] source/views/streaming/radio/controller.js

    • [ ] icon="ios-planet"
    • [ ] <ActionButton icon="ios-play" onPress={this.play} text="Listen" />
    • [ ] <ActionButton icon="ios-more" onPress={this.pause} text="Starting" />
    • [ ] <ActionButton icon="ios-pause" onPress={this.pause} text="Pause" />
    • [ ] <ActionButton icon="ios-bug" onPress={noop} text="Error" />
  • [x] modules/navigation-buttons/share.js const iconName = Platform.OS === 'ios' ? 'ios-share-outline' : 'md-share'-outline no longer exists; need to find alternative for iOS.

  • [ ] modules/food-menu/food-item-row.js const specialsIcon = Platform.OS === 'ios' ? 'ios-star' : 'md-star'

  • [ ] modules/filter/filter-toolbar-button.js ios: 'ios-arrow-down',

  • [ ] modules/filter/active-filter-button.js ios: 'ios-close-circle',

  • [ ] source/views/streaming/radio/buttons.js icon={Platform.OS === 'ios' ? 'ios-call' : 'md-call'} icon={Platform.OS === 'ios' ? 'ios-calendar' : 'md-calendar'}

  • [ ] source/views/stoprint/components/notice.js name={Platform.OS === 'ios' ? 'ios-print' : 'md-print'}

  • [ ] source/views/stoprint/components/error.js ios: 'ios-bug',

We should update them to use the new helper.

hawkrives avatar Feb 04 '19 02:02 hawkrives

@hawkrives I have a branch with work done on almost all of these. I will work on that in the short-term.

rye avatar Feb 04 '19 04:02 rye

(I'll work on fixing this issue some more this weekend, for sure. It's on my shortlist now.)

rye avatar Feb 22 '19 13:02 rye

Closed by #3598

drewvolz avatar Mar 26 '19 14:03 drewvolz

Every item in the checklist in the OP still needs to be updated to use the helper introduced in #3459.

hawkrives avatar Mar 26 '19 14:03 hawkrives

My bad. Thanks for reopening. @rye I've merged your PR and updated the checkboxes above 👍

drewvolz avatar Mar 26 '19 18:03 drewvolz

I want to double-check that this is complete now

hawkrives avatar Jan 21 '23 05:01 hawkrives