-
Notifications
You must be signed in to change notification settings - Fork 13.4k
feat(modal): add expandToScroll property to allow scrolling at all breakpoints #30097
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
Merged
Changes from 1 commit
Commits
Show all changes
45 commits
Select commit
Hold shift + click to select a range
ea68986
feat(modal): added snapBreakpoints to sheet modals
kumibrr 4970471
fix: removed trailing semicolon
kumibrr 95c5314
chore: ran build
kumibrr 1dcf5c9
refactor(modal): replaced snapBreakpoints array implementation with s…
kumibrr 96c5e41
feat(modal): add contentAnimation support and animateContentHeight op…
kumibrr 4d0d56f
Merge branch 'ionic-team:main' into main
kumibrr 37c5832
feat(modal): added footer animation.
kumibrr 566d3db
feat(modal): improved implementation
kumibrr 9ab3733
fix: added back breakpoint: 0
kumibrr 556cb6a
removed console.log
kumibrr ad90cd9
feat: added footer slide-in and slide-out animation
kumibrr 396bf73
fix: footerAnimation did not exist at gestures initialization.
kumibrr ac11277
restored footer onMove and onEnd animations
kumibrr 44b8152
refactor(modal): use a clone footer to prevent flickering
thetaPC 804d043
fix(modal): add animation when scrollAtEdge is false
thetaPC 59dd5b0
Merge pull request #1 from thetaPC/scroll
kumibrr 4052a86
feat(modal): implement footer visibility swap for scrollAtEdge handling
kumibrr f4cf4c1
chore: added comments explaining footer visibility swaps
kumibrr df96686
feat(modal): added footer visibility swap based on scrollAtEdge in le…
kumibrr 991e02a
fix(modal): padding value was always zero
kumibrr 7c24b1f
chore(modal): minor fixes
kumibrr 6d3473f
Merge remote-tracking branch 'upstream/main'
thetaPC 6778e42
feat(modal): add padding to the first toolbar in modal sheets
kumibrr cef6fd7
fix(modal): limited padding-top to only apply on ios styles
kumibrr c8392bb
chore(modal): added explanation for padding-top
kumibrr dc4caaf
chore(modal): added section for sheet modal
kumibrr f6702a7
chore(vue): remove file
thetaPC b1726ec
Merge branch 'main' of github.com:kumibrr/ionic-framework
thetaPC 42aa703
fix(modal): add missing parameter
thetaPC 30e2d8b
test(modal): update snapshots
thetaPC 3a79743
refactor(modal): add comments
thetaPC aca5855
Merge branch 'feature-8.5' into main
brandyscarney 6e07a6f
refactor(modal): add requested changes
thetaPC 80a826e
Merge branch 'main' of github.com:kumibrr/ionic-framework
thetaPC df2c331
chore(vue): run build
thetaPC e15ae91
fix(vue): add missing new line
thetaPC 4ffa5f6
fix(modal): add footer check
thetaPC bbbdb87
chore(core): run build
thetaPC 433f563
Update core/src/components/modal/animations/md.leave.ts
thetaPC a9ea2f4
Update core/src/components/modal/animations/ios.leave.ts
thetaPC c099027
Update core/src/components/modal/gestures/sheet.ts
thetaPC ce909b0
Update core/src/components/modal/gestures/sheet.ts
thetaPC c0d75b5
Update core/src/components/modal/gestures/sheet.ts
thetaPC 6416ebe
Update core/src/components/modal/gestures/sheet.ts
thetaPC f6cc539
Update core/src/components/modal/gestures/sheet.ts
thetaPC File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
fix(vue): add missing new line
- Loading branch information
commit e15ae91680d3a19065a293b9071f4277601210fd
There are no files selected for viewing
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
Oops, something went wrong.
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.
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.
Recommended by the Stencil team to delete this file in order to get the build to pass.