-
Improvement
-
Resolution: Deferred
-
Minor
-
None
-
4.1
-
MOODLE_401_STABLE
-
MDL-75681_refactor-bootstrap-5.x-data-attributes -
Fails against automated checks. 
Checked
MDL-75681using repository: https://github.com/ziegenberg/moodle
Testing instructions are missing.
main (0 errors / 5 warnings) [branch: MDL-75681_refactor-bootstrap-5.x-data-attributes | CI Job]
Built on: Tue 19 Nov 2024 07:50:41 PM UTC
Fails against automated checks. Checked MDL-75681 using repository: https://github.com/ziegenberg/moodle Testing instructions are missing. main (0 errors / 5 warnings) [branch: MDL-75681_refactor-bootstrap-5.x-data-attributes | CI Job ] overview (0/0) , phplint (0/0) , phpcs (0/0) , js (0/0) , css (0/0) , phpdoc (0/0) , commit (0/0) , savepoint (0/0) , thirdparty (0/0) , externalbackup (0/0) , grunt (0/0) , shifter (0/0) , mustache (0/5) , gherkin (0/0) , Should these errors be fixed? Built on: Tue 19 Nov 2024 07:50:41 PM UTC
Some bootstrap html data attributes have changed syntax causing UI component to stop working. Popovers are an example of these.
The old syntax on a button with some popover text:
<button type="button" class="btn btn-secondary" data-container="body" data-toggle="popover" data-placement="top" data-content="Top popover"> |
Popover on top
|
</button> |
The new syntax:
<button type="button" class="btn btn-secondary" data-bs-container="body" data-bs-toggle="popover" data-bs-placement="top" data-bs-content="Top popover"> |
Popover on top
|
</button> |
Find all uses of these popovers and update the html
