Skip to content

Popup javascript does not fire as expected due to Bootstrap 5 changes - #238

Open
Syxton wants to merge 2 commits into
gjbarnard:mainfrom
Syxton:moodle5-popup-fixes
Open

Popup javascript does not fire as expected due to Bootstrap 5 changes#238
Syxton wants to merge 2 commits into
gjbarnard:mainfrom
Syxton:moodle5-popup-fixes

Conversation

@Syxton

@Syxton Syxton commented Jul 9, 2025

Copy link
Copy Markdown

These are the changes I made to get the popup setting to work again in Moodle 5.x

gjb2048 and others added 2 commits July 9, 2025 11:18
due to bootstrap 5 changes
@Syxton Syxton mentioned this pull request Jul 9, 2025
@gjb2048

gjb2048 commented Jul 9, 2025

Copy link
Copy Markdown
Collaborator

@Syxton Not sure why the CSS attribute is needed and why it is inline rather than in the styles file -> 22e7ac3#diff-9dec2f9b87bcd30f64bf08a6c119fa45d03cfeeac0d8fb9dfb1dcf95f3bdf628L152.

@Syxton

Syxton commented Jul 9, 2025

Copy link
Copy Markdown
Author

@gjb2048 It wasn't related to the bug...it was just added while I was fixing what I was seeing. The X button on the modal was bunched up against the section title. Looks more natural in the right corner.

you are right. adding:
.format-grid .modal-header {
justify-content: space-between;
}

to styles.css is better, I shouldn't have added that in the pull

@gjb2048

gjb2048 commented Jul 9, 2025

Copy link
Copy Markdown
Collaborator

@Syxton Interesting! Looks like happens in Boost but not Adaptable (where I was testing), which has:

.modal-header .close {
  margin-left: auto;
}

@Syxton

Syxton commented Jul 9, 2025

Copy link
Copy Markdown
Author

Looks and behaves correctly with the new commit including the style change. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants