Skip to content
This repository has been archived by the owner on Feb 19, 2022. It is now read-only.

Multi line legend #246

Merged
merged 5 commits into from
May 16, 2017
Merged

Multi line legend #246

merged 5 commits into from
May 16, 2017

Conversation

angelanicholas
Copy link
Contributor

@angelanicholas angelanicholas commented May 16, 2017

  • fixes add VictoryLegend row/column wrapping logic victory#476
  • adds an optional itemsPerRow prop for displaying legend items in multiple rows/columns (when orientation is horizontal/vertical, respectively)
  • updates horizontal legend to use maxItemWidth for column width (as displayed in the demo screenshot below)

screen shot 2017-05-15 at 8 20 36 pm

@angelanicholas angelanicholas requested a review from boygirl May 16, 2017 02:58
@boygirl
Copy link
Contributor

boygirl commented May 16, 2017

@angelanicholas awesome! Approved!

@angelanicholas angelanicholas merged commit a92400b into master May 16, 2017
@angelanicholas angelanicholas deleted the multi-line-legend branch May 16, 2017 21:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add VictoryLegend row/column wrapping logic
2 participants