Skip to content

Commit

Permalink
fix: adjust container in the popup
Browse files Browse the repository at this point in the history
  • Loading branch information
hirsch88 committed Oct 17, 2023
1 parent 7cf3a84 commit 678ec75
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 4 deletions.
15 changes: 15 additions & 0 deletions packages/components/src/components/bal-popup/bal-popup.sass
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,21 @@
visibility: hidden
padding-top: var(--bal-popup-container-padding-top)
padding-bottom: var(--bal-popup-container-padding-bottom)
flex-grow: 1
margin: 0 auto
position: relative
width: auto
width: 100% !important
padding-left: var(--bal-container-space)
padding-right: var(--bal-container-space)
+tablet
padding-left: var(--bal-container-space-tablet)
padding-right: var(--bal-container-space-tablet)
+desktop
padding-left: var(--bal-container-space-desktop)
padding-right: var(--bal-container-space-desktop)
max-width: var(--bal-container-size-normal)


//
// variants
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@ export class DrawerVariantRenderer extends AbstractVariantRenderer implements Po
component.containerEl.style.setProperty('inset', `auto 0px 0px 0px`)
component.containerEl.style.setProperty('bottom', `${this.offset}px`)
component.backdropEl.style.setProperty('bottom', `${this.offset}px`)
component.containerEl.classList.add('container')

this.showBackdropElement(component, this.backdrop)
this.showContainerElement(component)
Expand All @@ -34,7 +33,6 @@ export class DrawerVariantRenderer extends AbstractVariantRenderer implements Po
this.hideContainerElement(component)

component.containerEl.style.removeProperty('inset')
component.containerEl.classList.remove('container')
component.containerEl.style.removeProperty('bottom')
component.backdropEl.style.removeProperty('bottom')
return true
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ export class FullscreenVariantRenderer extends AbstractVariantRenderer implement
this.showContainerElement(component)
this.showArrowElement(component)

component.containerEl.classList.add('container')
return true
}
return false
Expand All @@ -34,7 +33,6 @@ export class FullscreenVariantRenderer extends AbstractVariantRenderer implement
this.hideArrowElement(component)
component.containerEl.style.removeProperty('box-shadow')
component.containerEl.style.removeProperty('inset')
component.containerEl.classList.remove('container')
return true
}
return false
Expand Down

0 comments on commit 678ec75

Please sign in to comment.