mahara-contributors team mailing list archive

[Mahara Bot <1939165@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://reviews.mahara.org/11918
Committed: https://git.mahara.org/mahara/mahara/commit/607496dced78e2780fe3bd9bd45cbe345a43988f
Submitter: Robert Lyon (robertl@xxxxxxxxxxxxxxx)
Branch:    master

commit 607496dced78e2780fe3bd9bd45cbe345a43988f
Author: Evonne <evonne@xxxxxxxxxxxxxxx>
Date:   Wed Aug 11 13:51:22 2021 +1200

Bug 1939165 Bootstrap4 carousel styling

Aligning gallery slideshow carousel styling with bootstrap 4's example
and to make it accessible, including the caption which would appear
under the image instead of on top of the image.

behatnotneeded

Change-Id: I0ec6268a1e64b5ec25833314beeb8fa1ddd7c857

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: mahara-contributors
https://bugs.launchpad.net/bugs/1939165

Title:
  Bootstrap4 carousel doesn't render correctly

Status in Mahara:
  Fix Committed

Bug description:
  I tried to put a carousel into index.tpl using the example from the
  bootstrap documentation[1] but it's broken in a number of ways; the
  arrows and indicators don't render in the correct position and the
  animation does not run as expected and instead ends up as a sort of
  awkward shuffle (I can post an example if requested).

  I've narrowed down the problem to a number of overrides in the raw
  theme in Mahara which my theme is based on. If I remove the
  _carousel.scss from components/ and the carousel related code from
  _bootsrap-variables.scss and theme.js then the carousel works as
  expected. [2-4]

  A search through the commit log shows some fairly old commits related
  to carousel and image gallery.[5]

  [1] https://getbootstrap.com/docs/4.3/components/carousel/#with-indicators
  [2] https://git.mahara.org/mahara/mahara/-/blob/master/htdocs/theme/raw/sass/components/_carousel.scss
  [3] https://git.mahara.org/mahara/mahara/-/blob/master/htdocs/theme/raw/sass/utilities/_bootstrap-variables.scss#L1063
  [4] https://git.mahara.org/mahara/mahara/-/blob/master/htdocs/theme/raw/js/theme.js#L91
  [5] https://git.mahara.org/mahara/mahara/-/commits/master?search=carousel

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1939165/+subscriptions