-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
fix(core): actionBar to use appearance api on ios13+ #9530
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
Conversation
@farfromrefug should be fine because #9223 only touches android side, this only affects ios. I did change the base to |
@rigor789 that s what i thought at first but i think |
Is there any plan to roll this to and 8.0.x release? It is likely the new iOS / Xcode updates will be live in a month. Right now, I am not seeing a time table on the 8.1.0 release. It is fairly unrealistic to force developers to upgrade to the next version of nativescript in such a short amount of time. |
Once |
Thanks for the response Rigor. Back porting for us is critical. It is a rather large application and each release has 3 rounds of regression lasting a week each. In this time we are at code freeze and cannot do a port until QA has signed off on the release. |
@bmacdonald we just backported it to a If you run into any issues with XCode 13 do let us know asap so we can work on a fix before it goes stable. |
Thank you Rigor. I will test it either later this evening or in the morning tomorrow and get back with you. |
@rigor789 ✔ Getting NativeScript components versions information... Steps:
You will notice the title bar color stays at black. I think this has to do with the setColor property and changing the color of the text. We can bandaid it by doing something like this:
However, this should be fixed at the framework level. Let me know if you need any more information or testing. |
@bmacdonald thanks for this additional detail, we'll work that into a 8.0.10 patch update by early next week. |
@NathanWalker |
@bmacdonald |
@NathanWalker However, I have noticed another issue. If you have an action item, you can't set the color. Using my same steps above, change the action bar code to this:
You will notice the icon to the right is blue. It should be white. If you try to add a style property for color 'style="color:white"' the color does not change. Before if you set the actionbar color, the buttons / actionitems would match the color you set on the actionbar. |
Looks like you might need to add back tintColor for both legacy and new style. 'navBar.tintColor = color.ios;' |
@bmacdonald We just published |
@NathanWalker Are you planning on doing an 8.0.11? I've had several dev looking around to see if any other issues came up. So far, aside from the header issue, everything else appears to be working. Is the 8.1.0 upgrade going to be a major upgrade like 7 and 8 (huge webpack changes etc)? |
Yes this is published along with 8.0.11 now. https://github.com/NativeScript/NativeScript/releases/tag/8.0.11-core |
8.1 update is just bug fixes and stabilization primarily so easy update. |
Thank you! If you have an update script to point at the RC packages to update, I'd be more than happy to take a look before it's released. I could probably throw a couple of dev at it to take a look. |
@bmacdonald You can try the following anytime to have a look:
Would absolutely appreciate you having a look against your app and let us know if anything jumps out. |
I wanted to give you an update. I've had several developers testing the packages for 8.1 rc. I noticed the 8.1 cli was released so I had everyone update to that version. We have not found any issues with the 8.1 rc so far. We will continue to test for the rest of the day. If you don't hear back, then we have not found any issues. Also, it looks like the 8.0.11 release is not available on NPM. Thank you for your time. @NathanWalker @rigor789 |
Thank you @bmacdonald for update - I'll double check 8.0.11 and great to hear about 8.1! |
8.0.11 is confirmed up now. We'll be publishing 8.1 tomorrow. |
Awesome! We are about to start our regression testing. It will be a good time to sneak an upgrade in. |
PR Checklist
What is the current behavior?
What is the new behavior?
Fixes/Implements/Closes #[Issue Number].