mahara-contributors team mailing list archive
-
mahara-contributors team
-
Mailing list archive
-
Message #26803
[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