-
Notifications
You must be signed in to change notification settings - Fork 1.2k
[DropZone] Constrain drop zone height based on inherited wrapper height #908
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
a0f1613
to
725b2e5
Compare
725b2e5
to
c061c07
Compare
elileto
commented
Jan 21, 2019
c061c07
to
f38e5cc
Compare
f38e5cc
to
89c4a39
Compare
89c4a39
to
a35acb3
Compare
a35acb3
to
f3cf134
Compare
f3cf134
to
6ebd694
Compare
6ebd694
to
e856ab4
Compare
0bc7451
to
e4f1b6b
Compare
0f21fe1
to
b74ae3d
Compare
b74ae3d
to
0a8dd5b
Compare
6b84f72
to
59de6c8
Compare
solonaarmstrong-zz
approved these changes
Feb 19, 2019
d021b30
to
3e8f57b
Compare
AndrewMusgrave
approved these changes
Feb 20, 2019
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
3e8f57b
to
a45f23f
Compare
534d34c
to
c4a52d4
Compare
Merged
3 tasks
3 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
WHY are these changes introduced?
Resolves #742
The
DropZone
determines its size (height and width) based on width only. This makes certain valid use cases, where the defaultDropZone
height isn't appropriate, impossible.WHAT is this pull request doing?
This pull request allows the consumer to set a fixed height on a wrapping parent element so that the
DropZone
takes its parent's height if one is provided.Current Behaviour: (wrapped with a div given a height of 50px)

New Behaviour: (wrapped with a div given a height of 50px)

Fixed small Height small Width:






Fixed Medium Height Medium Width:
Fixed Large Height Large Width:
Fixed ExtraLarge Height ExtraLarge Width:
Fixed Small Width ExtraLarge Height
Fixed ExtraLarge Width Small Height
(Note: there are in between size variations (ie. small width, medium height etc.) The
FileUpload
will always render the smaller measurement of width or height.Default Width and Heights:

Fixed Small Height No fixed width:

Fixed Medium Height No fixed Width:

Fixed Large Height no Fixed width:

Fixed Extra Large Height no Fixed Width:

How to 🎩
🖥 Local development instructions
🗒 General tophatting guidelines
📄 Changelog guidelines
Code Example with small width, small height
medium width medium height :
Large width, large height
ExtraLarge width ExtraLarge height
Default width and height
Small width, Large Height
Large Width Small Height
🎩 checklist