prscX / react-native-siri-wave-view

React Native: Native Siri Wave View

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Wave new version of ios9.

Angelk90 opened this issue · comments

@prscX :
Hi, it would be possible to have the same effect in the images you see below in the new version of ios9.
screenshot
screenshot

Thanks @Angelk90 for raising the request.

Initially I had plans of integrating Siri Wave for iOS 9 as well, but I couldn't find any native library providing the same.

The library which you are referring to is a JS WebView based and we may end up in performance issues. However if you come across any such native library please feel free to raise PR or enhancement request for the same.

Thanks
Pranav

Hello
Recently, I've developed native module for iOS 9 Siri Wave.
Could u merge this with yours?

Sure, Definitely. Can you please share the link of your module, I will integrate this on priority :smile, I am not able to find in your repositories.

Many Thanks for the module @GreatPotter.

Thanks
Pranav

Hi @prscX, there have been news about this module on the new version of ios9?

Not Yet @Angelk90. Once @GreatPotter publish iOS 9 Siri we will integrate the same.

Thanks
Pranav

Hi @prscX
https://github.com/GreatPotter/SiriwaveIOS9
You could see my module there. I used your code basics.

Thanks @GreatPotter for sharing the repo.

I have check and it is working as expected. Please find the screenshot below:

screen shot 2018-03-27 at 3 32 13 pm

Can you please help me in order to understand the source implementation which you have made in order to achieve this:

  • Which classes contains your iOS9 wave implementation changes? Is it in SCSiriWaveformView.m or SiriWaveCurve.m class?

  • Are you planning to open source native iOS 9 wave implementation? or should I go forward and open source both?

Thanks
Pranav

I modified SCSiriWaveformView.m file and added
SiriWaveCurve.m, SiriWaveCurve.h files

@GreatPotter: So should I integrate those changes in this library? If so we can plan to Open Source two libraries:

  • Native iOS 9 Siri Wave
  • Integrate Native iOS 9 Siri Wave into this library.

Please let me know your thoughts about the same.

Thanks
Pranav

Yes, I used Open Source Native libraries and added custom functionality.

Hi @GreatPotter, Can you please provide the list of Open Source libraries which you have used in order to achieve this.

Thanks
Pranav

I used the same library as you used.

@GreatPotter: So the same library you have modified in order to achieve this? If so can I Open Source it by crediting it on your name?

Thanks
Pranav

Sure.

Thanks @GreatPotter.

Thanks
Pranav

Thanks
Maik

Hi @GreatPotter,

Many Thanks for the Siri Waveform 9 Implementation, I have integrated your implementation with this library. Now you will be able to use it by providing type prop as 1.

Hi @Angelk90,

With the help of @GreatPotter, I have enabled support for iOS Siri Wave 9. In order to use please install latest version 0.0.4 for the same.

V0.0.4

Please let me know in case you face any issues.

Thanks
Pranav

Hi @prscX

I hope collaborate with you at any time.

Thanks.
Maik

Definitely @GreatPotter :smile. Please feel free to reach at any time.

Thanks
Pranav

Hi @prscX
It's great to see you again.
Recently, I changed my account name to k1115.
Could you change GreatPotter to k1115 on readme, please?