github/reddit
1
1
Fork 0
mirror of https://github.com/reddit-archive/reddit.git synced 2026-01-30 01:08:32 -05:00
Code Issues Packages Projects Releases Wiki Activity

Have save/unsave button toggle back and forth

Browse Source 
This also removes the confirmation added in 3e74406
This commit is contained in:
Chad Birch
2013-02-24 21:53:39 -07:00
parent bc4a9e7c96
commit 676eb6fef6
2 changed files with 16 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
r2/r2/public/static/js/reddit.js
View File

@@ -202,12 +202,18 @@ function read_thing(elem) {
$.request("read_message", {"id": $(t).thing_id()});
}
function save_thing(elem) {
$(elem).thing().addClass("saved");
}
function unsave_thing(elem) {
$(elem).thing().removeClass("saved");
function toggle_save(elem) {
var form = $(elem).parents("form").first()
var next_text = form.find('[name="executed"]')
var text = next_text.val()
if ($(elem).thing().hasClass("saved")) {
change_state(elem, 'unsave', undefined, true)
} else {
change_state(elem, 'save', undefined, true)
}
$(elem).thing().toggleClass("saved")
next_text.val($(elem).text())
$(elem).text(text)
}
function click_thing(elem) {

10
r2/r2/templates/printablebuttons.html
View File

@@ -186,10 +186,9 @@
%if thing.is_loggedin:
<li>
%if thing.saved:
${ynbutton(_("unsave"), _("unsaved"), "unsave", "unsave_thing")}
${self.state_button("unsave", _("unsave"), "toggle_save(this)", _("save"))}
%else:
${self.state_button("save", _("save"), \
"return change_state(this, 'save', save_thing);", _("saved"))}
${self.state_button("save", _("save"), "toggle_save(this)", _("unsave"))}
%endif
</li><li>
%if thing.hidden:
@@ -261,12 +260,11 @@
</li>
%if thing.saved:
<li class="comment-unsave-button">
${ynbutton(_("unsave"), _("unsaved"), "unsave", "unsave_thing")}
${self.state_button("unsave", _("unsave"), "toggle_save(this)", _("save"))}
</li>
%else:
<li class="comment-save-button">
${self.state_button("save", _("save"), \
"return change_state(this, 'save', save_thing);", _("saved"))}
${self.state_button("save", _("save"), "toggle_save(this)", _("unsave"))}
</li>
%endif
%if c.profilepage: