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 _______________________________________________ Mailing list: https://launchpad.net/~mahara-contributors Post to : [email protected] Unsubscribe : https://launchpad.net/~mahara-contributors More help : https://help.launchpad.net/ListHelp
