mahara-contributors team mailing list archive

[Jinelle Foley-Barnes <jinelleb@xxxxxxxxxxxxxxx>]

** Tags added: no-behat-needed

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1429871

Title:
  Link underlining in skins doesn't work

Status in Mahara ePortfolio:
  Fix Released
Status in Mahara 1.10 series:
  Fix Released
Status in Mahara 1.8 series:
  Fix Released
Status in Mahara 1.9 series:
  Fix Released
Status in Mahara 15.04 series:
  Fix Released
Status in Mahara 15.10 series:
  Fix Committed

Bug description:
  If the user edits a skin (Edit skin > Fonts and colours) and checks
  the Underline link -option, the link doesn't get underlined on a page
  with the skin applied. There seems to be a bug in skin/style.php#38
  with the comparison

  if ($option == 'transparent')

  because when a checkbox is selected, the $option value is a boolean
  TRUE and when using PHP's loose comparison operator, (true ==
  'transparent') = true. Therefore for example
  $skin->viewskin['view_link_normal_underline'] (and a few others) value
  changes to an empty string and the value assigned to template is
  'none' instead of 'underline'.

  A simple fix is to use strict comparison:

  if ($option === 'transparent')

  ---

  Mahara version: 1.9.1
  Browser: Chrome (Linux) 40.0.2214.115 (64-bit)

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