Summary

[WIP] Adding and removing widgets

Review Request #5676 — Created April 5, 2014 and discarded May 13, 2014, 9:44 p.m.

Information

Owner

hellosjsu

Repository

Review Board

Branch

master

Bugs

Depends On

Reviewers

Groups

reviewboard, students

People

Description

Users can add widgets to their admin dashboard by clicking on "Add Widgets," which will open a modal displaying a list of widgets that they can choose from.

Users can remove widgets by clicking the X in the upper right hand corner. A confirmation dialogue will appear to ask the user if they are sure.

Testing Done

Files

Issues

Description	From	Last Updated
the default parameter for dict.get defaults to None, so you don't need to include the second argument.	david	April 26, 2014, 2:53 a.m.
the default parameter for dict.get defaults to None, so you don't need to include the second argument.	david	April 26, 2014, 2:53 a.m.
This should probably check if widget is not None before assigning to positions.	david	April 26, 2014, 2:53 a.m.
Two blank lines between top-level items in the file.	david	April 26, 2014, 2:53 a.m.
Same comment re: get.	david	April 26, 2014, 2:53 a.m.
Same comment re: get.	david	April 26, 2014, 2:53 a.m.
Should test that widget_position isn't None.	david	April 26, 2014, 2:53 a.m.
Two blank lines.	david	April 26, 2014, 2:53 a.m.
Old code, but this should be two blank lines.	david	April 26, 2014, 2:53 a.m.
The parens don't really add anything here.	david	April 26, 2014, 2:53 a.m.
Remove the spaces inside of the parens.	david	April 26, 2014, 2:53 a.m.
This line should be 4-space indented.	david	April 26, 2014, 2:54 a.m.
Leftover debug code?	david	April 26, 2014, 2:54 a.m.
Remove the spaces inside the parens.	david	April 26, 2014, 2:54 a.m.
Remove the spaces inside the parens.	david	April 26, 2014, 2:54 a.m.
This line should be 4-space indented.	david	April 26, 2014, 2:54 a.m.
Remove the spaces inside the parens.	david	April 26, 2014, 2:54 a.m.
Remove the spaces inside the parens.	david	April 26, 2014, 2:54 a.m.
Alignment in HTML should use single spaces instead of tabs. Please check this for all your newly-added code here.	david	April 26, 2014, 2:54 a.m.
This should use {% trans "Manage Large Widgets" %} so that translators can localize it to other languages.	david	April 26, 2014, 2:54 a.m.
This should use {% trans "Manage Small Widgets" %} so that translators can localize it to other languages.	david	April 26, 2014, 2:54 a.m.
{% trans %}	david	April 26, 2014, 2:54 a.m.
This should use {% trans "Save Widgets" %} so that translators can localize it to other languages.	david	April 26, 2014, 2:54 a.m.
{% trans %}	david	April 26, 2014, 2:54 a.m.
{% trans %}	david	April 26, 2014, 2:54 a.m.

Description:

		Users can add or remove widgets to their admin dashboard by clicking on "Manage Widgets," which will open a modal displaying a list of widgets that they can choose from.
	+
	+	Note 4/5/13: Ignore the changes on views.py for now (They're from the drag & drop feature that's already been shipped; I'm not sure why they show up as new changes here.)

Added Files:

Groups:

+	students

Description:

		Users can add or remove widgets to their admin dashboard by clicking on "Manage Widgets," which will open a modal displaying a list of widgets that they can choose from.

~		Note 4/5/13: Ignore the changes on views.py for now (They're from the drag & drop feature that's already been shipped; I'm not sure why they show up as new changes here.)
	~	Note 4/5/13: Ignore the changes on views.py for now (They're from the drag & drop feature that's already been shipped; I messed up when rebasing on top of master.)