← Back to team overview

mahara-contributors team mailing list archive

[Bug 1378176] Re: Skins folder on upload of background images - multiple images attached to skins

 

What happens if two skins include a file called background.png? Deleting the image if it exists wouldn't be so good if that's a possible scenario ...
I do think it's a bit confusing to add images to the root directory, particularly if they have quite generic names. If skins are likely to be able to contain more images in the future it might even make sense to store them in "page-skins/skin-name/..."

On a related note, should there be an option to delete all associated
images when you delete a skin? For example if you're trying out various
different skins from the internet and deleting the ones you don't want,
it would be annoying to have to go through and delete all the related
images as well.

-- 
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/1378176

Title:
  Skins folder on upload of background images - multiple images attached
  to skins

Status in Mahara ePortfolio:
  Confirmed

Bug description:
  A feature request, although not exactly sure if a bug/issue
  If a skin is deleted, and a skin is re-imported, the background images are also re-imported.
  This can be visually confusing for users (see attached image: multiple_copies_images.png) as it creates multiple copies of the same image. I know the image files have unique background identifiers, its just that many users will get confused, especially if they are learning to use skins and go to delete a file and accidentally delete the one their skin is using.
  Is it possible to have a folder automatically created on upload of an image called 'page-skins' for background image purposes?
  Or have it so that it recognises the image is already there, maybe?

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


References