FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fixed a crash in action bar on iOS 9.3 by tzraikov · Pull Request #2262 · NativeScript/NativeScript · GitHub

Fixed a crash in action bar on iOS 9.3 - #2262

Merged
tzraikov merged 1 commit into
masterfrom
raikov/fix-actionbar
Jun 8, 2016
Merged

tzraikov merged 1 commit into
masterfrom
raikov/fix-actionbar

Conversation

tzraikov commented Jun 7, 2016

Copy link
Copy Markdown
Contributor

ns-bot added the cla: yes label Jun 7, 2016
for (var i = 0; i < items.length; i++) {
var barButtonItem = this.createBarButtonItem(items[i]);
let items = this.actionItems.getVisibleItems();
let leftBarItems = NSMutableArray.alloc().init();

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Use of JS array is recommended as runtimes will marshal it in an optimized way;

tzraikov force-pushed the raikov/fix-actionbar branch from 3d74677 to 5d54ae6 Compare June 8, 2016 08:01

Copy link
Copy Markdown
Contributor

👍

1 similar comment

Copy link
Copy Markdown
Contributor

👍

tzraikov merged commit d9800c1 into master Jun 8, 2016
tzraikov deleted the raikov/fix-actionbar branch June 8, 2016 15:44
hshristov added this to the 2.1 milestone Jul 1, 2016

lock Bot commented Aug 28, 2019

Copy link
Copy Markdown

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

lock Bot locked and limited conversation to collaborators Aug 28, 2019
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL