Rename the pre/post modal popup functions to be more generic
Bug #1564681 reported by
Robert Lyon
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Mahara |
Confirmed
|
Low
|
Unassigned | ||
Bug Description
In js/views.js we have some functions that were originally for hiding/showing media players.
Now that these functions are not just doing things for media players but can be used for more things we should rename them to be more generic to avoid confusion about what they are for.
It looks like they are only called in this file so changing them should be easy enough.
I'm thinking of changing:
hideMediaPlayers() -> preModalOpen()
showMediaPlayers() -> postModalClose()
To post a comment you must log in.
