Skip to content

Modal#123

Open
stevenyuen wants to merge 5 commits into
masterfrom
modal
Open

Modal#123
stevenyuen wants to merge 5 commits into
masterfrom
modal

Conversation

@stevenyuen

@stevenyuen stevenyuen commented Feb 6, 2018 •
edited
Loading

Copy link
Copy Markdown
Contributor
  • Modal close button is based on Modal component's z-index. Updated styling of close button
  • Ability to set percentage-based width and height
  • closeURL prop not required if onClose func provided
  • Ability to set Overlay Background color/opacity
  • Use React Portals
  • Updated Docs

@stevenyuen stevenyuen requested a review from asabhaney February 28, 2018 04:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@asabhaney asabhaney Awaiting requested review from asabhaney

Assignees

No one assigned

Labels

ready for review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@stevenyuen