Skip to content
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

Close modal make body scrollable #6683

Closed
frenzymind opened this issue Sep 16, 2024 · 1 comment
Closed

Close modal make body scrollable #6683

frenzymind opened this issue Sep 16, 2024 · 1 comment
Assignees

Comments

@frenzymind
Copy link

frenzymind commented Sep 16, 2024

Hello. May be it can be related with this bug getModalsCount issue

Bug description:

With nested modals, when you close upper one, it is make body scrollable again (remove class 'modal-open' from body and inline styles).
When modal it self have a content to scroll, it is impossible to scroll modal for now, and this is the issue.

Code that reproduces the issue:

Use example from official docs: Nested modals:

  1. press 'open first modal' button
  2. press 'open seconds modal' button
  3. Check that body have no scroll
  4. press 'close self'
  5. body have scroll from now

Versions of ngx-bootstrap, Angular, and Bootstrap:

ngx-bootstrap: 18.0.2

Angular: 18.2.2

Bootstrap: 4.6.2

Expected behavior

No scroll on body, when there is at least one opened modal exists.

@frenzymind
Copy link
Author

I have to manually check count of modals. My workaround:

const BODY_OPEN_MODAL_CLASS = 'modal-open';


@Injectable()
export class NgxCustomBsModalService extends BsModalService {
  private rendererFactory = inject(RendererFactory2);
  private renderer2 = this.rendererFactory.createRenderer(null, null);

  override hide(id?: number | string): void {

    this._hideBackdrop();
    this._hideModal(id);

    const bodyEl = document.querySelector('body')!;
    const openModalsEl = document.querySelectorAll('modal-container');

    if (openModalsEl.length > 0) {
      if (bodyEl.classList.contains(BODY_OPEN_MODAL_CLASS) === false) {
        this.renderer2.addClass(bodyEl, BODY_OPEN_MODAL_CLASS);
      }
    } else {
      this.renderer2.removeClass(bodyEl, BODY_OPEN_MODAL_CLASS);
      this.renderer2.removeStyle(bodyEl, 'padding-right');
      this.renderer2.removeStyle(bodyEl, 'overflow-y');
    }
  }
}

and in my app module file:

  providers: [
    // ...
    { provide: BsModalService, useClass: NgxCustomBsModalService },
  ],
})
export class AppModule {}

@lexasq lexasq self-assigned this Oct 22, 2024
@lexasq lexasq mentioned this issue Oct 22, 2024
5 tasks
@lexasq lexasq closed this as completed Oct 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants