github/reddit
1
1
Fork 0
mirror of https://github.com/reddit-archive/reddit.git synced 2026-02-04 03:35:09 -05:00
Code Issues Packages Projects Releases Wiki Activity

Make the subreddit field required.

Browse Source
This commit is contained in:
Max Goodman
2012-12-05 13:36:43 -08:00
committed by Neil Williams
parent 09b0be2583
commit 8bf881535f
2 changed files with 7 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
r2/r2/templates/newlink.html
View File

@@ -100,7 +100,7 @@ ${thing.formtabs_menu}
<div class="spacer">
<%utils:round_field title="${_('choose a subreddit')}" id="reddit-field">
${reddit_selector(thing.default_sr.name if thing.default_sr else None, thing.sr_searches, thing.subreddits)}
${reddit_selector(thing.default_sr.name if thing.default_sr else None, thing.sr_searches, thing.subreddits, required=True)}
</%utils:round_field>
</div>

8
r2/r2/templates/utils.html
View File

@@ -523,7 +523,7 @@ ${unsafe(txt)}
</tr>
</%def>
<%def name="reddit_selector(default_sr, sr_searches, subreddits)">
<%def name="reddit_selector(default_sr, sr_searches, subreddits, required=False)">
<%doc>
Fires custom event when subreddit selection changes. Add handler like:
$("#sr-autocomplete").bind("sr-changed", function() { dostuff; })
@@ -535,7 +535,11 @@ ${unsafe(txt)}
onkeyup="sr_name_up(event)"
onkeydown="return sr_name_down(event)"
onblur="hide_sr_name_list()"
value = "${default_sr}" />
value = "${default_sr}"
% if required:
required
% endif
/>
<ul id="sr-drop-down">
<li class="sr-name-row"
onmouseover="highlight_reddit(this)"