Skip to content

Conversation

@nicolaskruchten
Copy link
Contributor

Fixes #436

<NumericFractionInverse label={_('Box Width')} attr="boxgap" />
<NumericFraction label={_('Bar Padding')} attr="bargroupgap" />
<NumericFraction label={_('Box Padding')} attr="boxgroupgap" />
</LayoutSection>
Copy link
Member

Choose a reason for hiding this comment

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

This abstraction / API is waaaay better. It looks so freaking obvious now. All the good fixes do. Made my day!

@nicolaskruchten
Copy link
Contributor Author

@VeraZab this is now waiting on your renaming/visibility stuff and then I can 🔪 the rest of the supplyPlotProps machinery

}
};

export const shamefullyAdjustGeo = ({layout: {geo = {}}}, {update}) => {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@VeraZab this is finally how I got the geo stuff sorted. I also had to deal with the projection centers etc. Note that setting the value to {} essentially clears it, so this restores the defaults instead of hardcoding mappings, which is nice :)

@nicolaskruchten
Copy link
Contributor Author

Hmmm now that I look at it, it may be easier to merge my change in first... I could help you rebase :)

@VeraZab
Copy link
Contributor

VeraZab commented Mar 28, 2018

yeah, looks like it'd be easy to rebase this one..
well, looks good to me, I'll look a little closer at the code tomorrow, but looks ready for a merge
thanks for the api fixes!

@nicolaskruchten nicolaskruchten changed the title WIP 🔪 supplyPlotProps 🔪 supplyPlotProps Mar 28, 2018
@VeraZab
Copy link
Contributor

VeraZab commented Mar 28, 2018

💃

@nicolaskruchten nicolaskruchten merged commit d278280 into master Mar 28, 2018
@nicolaskruchten nicolaskruchten deleted the supplyPlotPropsHocho branch March 28, 2018 15:01
@nicolaskruchten nicolaskruchten restored the supplyPlotPropsHocho branch July 18, 2018 17:16
@nicolaskruchten nicolaskruchten deleted the supplyPlotPropsHocho branch July 18, 2018 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants