Page themes don't work with configurable theme

Bug #1607743 reported by Kristina Hoeppner
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Medium
Unassigned

Bug Description

Master mahara (but also on 16.04 and 15.10)

When users can choose a theme for their pages and the institution uses the configurable theme, nothing is being displayed properly.

Part of the configurable theme is always visible but the acutal chosen theme is used for the navigation, which does not look good. And then putting skins on top...

To replicate the issue:

1. Allow users to choose page themes: Admin -> Configure site -> User settings -> Users can choose page themes.
2. Create an institution and choose the configurable theme. Choose a dark background color so that header nav items need to be white.
3. Add a user to that institution.
4. Set up a page as that user and give it a different theme.
Expected result: The theme overwrites the configurable theme completely like it does when a an institution has a regular theme.
Actual result: Part of the configurable theme is still visible (see screenshot).

The screenshot also shows how it would look like with a skin. The skin doesn't influence it badly though. Just something to keep in mind for testing.

Tags: front-end
Revision history for this message
Kristina Hoeppner (kris-hoeppner) wrote :
Revision history for this message
Kristina Hoeppner (kris-hoeppner) wrote :

By the looks of things, the theme is loaded first but then the configurable theme is loaded after that. We will most likely need to change that order in the code.

Robert Lyon (robertl-9)
Changed in mahara:
milestone: 16.10.0 → 16.10.1
Robert Lyon (robertl-9)
Changed in mahara:
milestone: 16.10.1 → 17.04.0
Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "master" branch: https://reviews.mahara.org/7215

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/7215
Committed: https://git.mahara.org/mahara/mahara/commit/961ae5443285503f0d3358bb993fad175ed15c1f
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit 961ae5443285503f0d3358bb993fad175ed15c1f
Author: Pat Kira <email address hidden>
Date: Fri Oct 28 16:42:30 2016 +1300

Bug 1607743 Fix page theme comflicting with configuarable theme

When configurable theme is set as well as page theme.
The page displays a mix of styles (configurable styles and page styles).

We only want the page to have the specified page theme.

Change-Id: Ifa2e998d864c86df9ffb70dd68c0abcc52088e13
behatnotneeded: styling only

Robert Lyon (robertl-9)
Changed in mahara:
status: Confirmed → Fix Committed
Robert Lyon (robertl-9)
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.