Move all pop-ups to a common view
Bug #1182483 reported by
Erik Wikström
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| LenaSYS |
Fix Committed
|
Undecided
|
Erik Wikström | ||
Bug Description
Pop-ups clot up the areas from where they are spawned. A menu that makes a pop-up appear does not need to have the pop-up itself inline. Moving all pop-ups to the same place and fetching them from there would be a cleaner way to handle them.
Related branches
lp:~wikxen/lenasys/popups
- Gustav Hartvigsson: Approve
-
Diff: 885 lines (+246/-472)17 files modifiedcodeigniter/application/controllers/ajax.php (+8/-0)
codeigniter/application/models/popups.php (+0/-136)
codeigniter/application/views/header.php (+1/-44)
codeigniter/application/views/manageCoursesBody.php (+0/-42)
codeigniter/application/views/popup/addCategory.php (+9/-0)
codeigniter/application/views/popup/addCourse.php (+18/-0)
codeigniter/application/views/popup/addNewInstructor.php (+12/-0)
codeigniter/application/views/popup/addStudents.php (+13/-0)
codeigniter/application/views/popup/changeActiveCourse.php (+8/-0)
codeigniter/application/views/popup/editCourseDetails.php (+18/-0)
codeigniter/application/views/popup/loginForm.php (+14/-0)
codeigniter/application/views/popup/passwordChanger.php (+23/-0)
codeigniter/application/views/popup/subjectConfirmRemove.php (+8/-0)
codeigniter/css/header.css (+0/-169)
codeigniter/css/popup.css (+110/-2)
codeigniter/js/header.js (+0/-75)
codeigniter/js/popup.js (+4/-4)
| Changed in lenasys: | |
| status: | New → Confirmed |
| Changed in lenasys: | |
| assignee: | nobody → Erik Wikström (wikxen) |
| status: | Confirmed → In Progress |
| Changed in lenasys: | |
| status: | In Progress → Fix Committed |
To post a comment you must log in.
