Fix link color in Primary School and Ocean themes for "Notifications"

Bug #1756154 reported by Kristina Hoeppner on 2018-03-15
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
High
Unassigned
18.04
High
Unassigned
18.10
High
Liam

Bug Description

When you have the Primary School or Ocean theme on your site, the navigation bar in the inbox (/module/multirecipientnotification/inbox.php) does not show the links in the nav bar in the correct colors. In Ocean it is only the active link that is a problem, in Primary School it is both Inbox and Sent.

Changed in mahara:
milestone: none → 18.04.0
Jaque (jaque777) on 2018-03-15
Changed in mahara:
status: Confirmed → In Progress
Jaque (jaque777) wrote :

On my local instance, I can not replicate the problem on either the Ocean or the primary school theme, the colors are working correctly.

I can show you tomorrow, Jaque.

This only seems to happen when you upgrade a site.

Robert Lyon (robertl-9) wrote :

I do notice for the 'ocean' theme there is this on the sass/utilities/ dir:

_custom-variables.scss:$arrow-color: $brand-primary;
_custom-variables.scss:$arrow-text-color: #fff;
_custom-variables.scss:$arrow-link-color: #fff;
_custom-variables.scss:$arrow-link-hover-color: $text-color;
_custom-variables.scss:$arrow-link-active-color: #20738f;
_custom-variables.scss:$arrow-link-background-color: #fff;

_bootstrap-variables.scss:$arrow-color: darken($brand-info, 10%);
_bootstrap-variables.scss:$arrow-text-color: #fff;
_bootstrap-variables.scss:$arrow-link-color: #fff;
_bootstrap-variables.scss:$arrow-link-hover-color: $arrow-color;
_bootstrap-variables.scss:$arrow-link-active-color: $arrow-color;

And it looks like some things are defined twice and with different values

--
In primary school theme

we have
_custom-variables.scss:$arrow-color: $brand-primary;
_custom-variables.scss:$arrow-text-color: #fff;
_custom-variables.scss:$arrow-link-color: #ca0238;
_custom-variables.scss:$arrow-link-hover-color: $text-color;
_custom-variables.scss:$arrow-link-active-color: $brand-primary;
_custom-variables.scss:$arrow-link-background-color: #fff;
_custom-variables.scss:$arrow-link-group-color: #799323;

and I notice _custom-variables.scss:$arrow-link-color: #ca0238; would give the 'Sent' button the red colour

Reviewed: https://reviews.mahara.org/8682
Committed: https://git.mahara.org/mahara/mahara/commit/113faa354393da3480ddfaf2f3d46ea2abd15d52
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit 113faa354393da3480ddfaf2f3d46ea2abd15d52
Author: Liam <email address hidden>
Date: Thu Mar 22 10:20:08 2018 +1300

Bug #1756154: Fix link color in Primary School and Ocean themes for "Notifications"

behatnotneeded

Change-Id: I674cb8242cc1c522195647da5be4002440beafb8

Reviewed: https://reviews.mahara.org/8686
Committed: https://git.mahara.org/mahara/mahara/commit/e1ba168c6678116f543c4d72145a052c5140e35c
Submitter: Robert Lyon (<email address hidden>)
Branch: 18.04_STABLE

commit e1ba168c6678116f543c4d72145a052c5140e35c
Author: Liam <email address hidden>
Date: Thu Mar 22 10:20:08 2018 +1300

Bug #1756154: Fix link color in Primary School and Ocean themes for "Notifications"

behatnotneeded

Change-Id: I674cb8242cc1c522195647da5be4002440beafb8
(cherry picked from commit 113faa354393da3480ddfaf2f3d46ea2abd15d52)

Reviewed: https://reviews.mahara.org/8693
Committed: https://git.mahara.org/mahara/mahara/commit/57e9c6b1045bc843fa0bb5514e1b6d3e564138c5
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit 57e9c6b1045bc843fa0bb5514e1b6d3e564138c5
Author: Liam <email address hidden>
Date: Thu Mar 22 17:18:26 2018 +1300

Bug #1756154: Fix link color in Primary School and Ocean themes for "Notifications"

Made the notifications style the default style

behatnotneeded

Change-Id: I1da5f02918f75a6eb59cb059ff8f9941b78cc023

Reviewed: https://reviews.mahara.org/8707
Committed: https://git.mahara.org/mahara/mahara/commit/52217820c93e0fad5f8bd8a8260b5c38030d6737
Submitter: Robert Lyon (<email address hidden>)
Branch: 18.04_STABLE

commit 52217820c93e0fad5f8bd8a8260b5c38030d6737
Author: Liam <email address hidden>
Date: Thu Mar 22 17:18:26 2018 +1300

Bug #1756154: Fix link color in Primary School and Ocean themes for "Notifications"

Made the notifications style the default style

behatnotneeded

Change-Id: I1da5f02918f75a6eb59cb059ff8f9941b78cc023
(cherry picked from commit 57e9c6b1045bc843fa0bb5514e1b6d3e564138c5)

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers