-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove prototype from app/views/shared #598
Comments
general_accordion_nav can be removed: rigse/app/views/layouts/application.html.haml Lines 82 to 87 in b7c6dba
In those 3 cases it not necessary anymore. It is OK in those cases to fall through to the 'portal_teacher' and 'portal_student' cases. |
It looks like I'm pretty sure the sortable_element part of offering_for_teacher can be removed. I believe this partial is only rendered on an admin page here: |
It also looks like |
I suspect |
To summarize I believe all these partials and views can either be removed, or their dynamic (sortable draggable) elements can be removed. I'll mark it as dead for that reason. |
Ref #457
app/views/shared/_accordion_nav.html.haml:34 #sortable_element
app/views/shared/_accordion_nav.html.haml:35 #sortable_element
app/views/shared/_activity_trail.html.haml:27 #link_to_function
app/views/shared/_general_accordion_nav.html.haml:62 #sortable_element
app/views/shared/_general_accordion_nav.html.haml:64 #sortable_element
app/views/shared/_offering_for_teacher.html.haml:26 #sortable_element
app/views/shared/_page_header.html.haml:4 #link_to_remote
uses CALLBACKS and AJAX_OPTIONS
app/views/shared/_runnable.html.haml:12 #draggable_element
app/views/shared/_runnables_listing.html.haml:36 #draggable_element
The text was updated successfully, but these errors were encountered: