React-native-navigation: Small button area in navbar on ios

Created on 3 Dec 2019  ·  16Comments  ·  Source: wix/react-native-navigation

Issue Description

Small button area in navbar on ios. With real use cases, it’s very difficult to hit the button.

Steps to Reproduce / Code Snippets / Screenshots

Снимок экрана 2019-12-03 в 16 44 15


Environment

  • React Native Navigation version: 4.0.3
  • React Native version: 0.61.5
  • Platform(s) (iOS, Android, or both?): iOS
  • Device info (Simulator/Device? OS version? Debug/Release?): All ios devices Debug, Release
TopBar Buttons iOS acceptebug

All 16 comments

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If you believe the issue is still relevant, please test on the latest Detox and report back. Thank you for your contributions.

not stale

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If you believe the issue is still relevant, please test on the latest Detox and report back. Thank you for your contributions.

Not stale

not stale. have issues with left button / back button as well

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If you believe the issue is still relevant, please test on the latest version and report back. Thank you for your contributions.

not stale

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If you believe the issue is still relevant, please test on the latest version and report back. Thank you for your contributions.

not stale lol

not stale.

FWIW - I have a post install patch that fixes the issue for me, its just annoying to have to make my own edits to this: ./node_modules/react-native-navigation/lib/ios/RNNUIBarButtonItem.m

-(instancetype)init:(NSString*)buttonId withIcon:(UIImage*)iconImage {
  // UIButton* button = [[UIButton alloc] init];
  // [button addTarget:self action:@selector(onButtonPressed) forControlEvents:UIControlEventTouchUpInside];
  // [button setImage:iconImage forState:UIControlStateNormal];
  // [button setFrame:CGRectMake(0, 0, iconImage.size.width + 40, iconImage.size.height + 20)];
  // self = [super initWithCustomView:button];
  // self.buttonId = buttonId;
  // button.contentHorizontalAlignment = UIControlContentHorizontalAlignmentLeft;
  self = [super initWithImage:iconImage style:UIBarButtonItemStylePlain target:self action:@selector(onButtonPressed)];
  self.buttonId = buttonId;
  return self;
}

@sorokin0andrey lmk if this works for you and hopefully if its legit the wix team can get this changed in the source

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If you believe the issue is still relevant, please test on the latest version and report back. Thank you for your contributions.

Not stale

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If you believe the issue is still relevant, please test on the latest version and report back. Thank you for your contributions.

Not stale

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If you believe the issue is still relevant, please test on the latest version and report back. Thank you for your contributions.

Duplicate of #5000 , PR at #6482

Was this page helpful?
0 / 5 - 0 ratings

Related issues

zagoa picture zagoa  ·  3Comments

viper4595 picture viper4595  ·  3Comments

charlesluo2014 picture charlesluo2014  ·  3Comments

birkir picture birkir  ·  3Comments

swingywc picture swingywc  ·  3Comments