react-native-graph icon indicating copy to clipboard operation
react-native-graph copied to clipboard

Add prop for fixed selection dot and fadeout starting point.

Open gabimoncha opened this issue 1 year ago • 12 comments

  1. Update C++ standard to c++17
  2. Update Cocoapods version to latest stable
  3. Add selectionDotValueX prop for a fixed position of the selection dot
  4. Add fadeouValueX prop for a fixed starting point for the fade out

https://github.com/margelo/react-native-graph/assets/39256258/e97f6e42-9701-4de5-b297-1dcc57d190bc

gabimoncha avatar Feb 02 '24 13:02 gabimoncha

@mrousavy would you be interested in merging this enhancement?

gabimoncha avatar Mar 12 '24 13:03 gabimoncha

Hey - yep, I am! I'm just waiting for @chrispader to get back from vacation, as he's the main responsible maintainer for RN Graph nowadays :)

mrousavy avatar Mar 12 '24 14:03 mrousavy

thanks for your PR on this!!

mrousavy avatar Mar 12 '24 14:03 mrousavy

Cool! Thanks for letting me know. Will continue any further discussion and ideas with him

gabimoncha avatar Mar 12 '24 14:03 gabimoncha

I have been using this, as well as @gabimoncha 's forked CustomIndicator component PR would be good to get these merged

The only ammendment would be the logic in the AnimatedLineGraph.tsx circa line 535, if an indicator component is used then the indicator pulsing is disabled it would be good to have both, just duplicating the indicatorPulsing into your ternary operator conditional works well enough, see below. I can submit a PR if needed gabi :)

 (IndicatorComponent != null ? (
                  <Group>
                    {indicatorPulsating && (
                       <Circle
                         cx={indicatorX}
                         cy={indicatorY}
                         r={indicatorPulseRadius}
                         opacity={indicatorPulseOpacity}
                         color={indicatorPulseColor}
                         style="fill"
                       />
                     )}
                  <IndicatorComponent
                    isActive={isActive}
                    color={color}
                    circleX={indicatorX}
                    circleY={indicatorY}
                  />
                  </Group>
 

theblondealex avatar Mar 27 '24 11:03 theblondealex

I'm gonna try to look into this issue (and others) on the weekend or next week. Unfortunately i'm fully booked with work right now...

chrispader avatar Mar 27 '24 12:03 chrispader

@chrispader let me know if you need any help to streamline this :D

gabimoncha avatar Mar 28 '24 12:03 gabimoncha

any chance we can get these merged? @chrispader as well as @gabimoncha's other forked repo containing the custom indicatordot?

theblondealex avatar Apr 11 '24 09:04 theblondealex

@theblondealex @chrispader - I have another one where I pass the index of the data point to onPointSelected in case you want to use it for additional visualisation in a larger dataset object 😅

gabimoncha avatar Apr 11 '24 12:04 gabimoncha

i'm gonna look into this tmrw! sorry for the delay...

@theblondealex @chrispader - I have another one where I pass the index of the data point to onPointSelected in case you want to use it for additional visualisation in a larger dataset object 😅

@gabimoncha could you provide me a link to that?

chrispader avatar Apr 15 '24 18:04 chrispader

@chrispader - here Each branch is a different feature for the app. The ones I told you on discord too

  1. Custom starting point for selection dot. this PR
  2. Custom Indicator - same props as SelectionDot.
  3. Passing the index in onPointSelected, if needed to link data point to a bigger data set

gabimoncha avatar Apr 16 '24 20:04 gabimoncha

@chrispader any updates on this PR? would love to push the other solutions too

gabimoncha avatar Apr 30 '24 17:04 gabimoncha