Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(3043)

Issue 111210043: Change private checkbox to protected and add help text (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years, 9 months ago by jrobbins (chromium)
Modified:
2 years, 4 months ago
Visibility:
Public.

Description

On the edit form, use the label "Protected" instead of "Private". To accommodate help text, space out all input fields from each other a little more, make the labels closer to the form elements (rather than the middle of the form element table cell). To emphasize the need to avoid mailing lists on protected issue, show that help text in bold and make it appear whenever the checkbox is checked. B=394013

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+32 lines, -7 lines) Patch
M codereview/views.py View 1 chunk +5 lines, -1 line 1 comment Download
M static/styles.css View 1 chunk +21 lines, -0 lines 0 comments Download
M templates/admin_update_stats.html View 1 chunk +1 line, -1 line 0 comments Download
M templates/block_user.html View 1 chunk +1 line, -1 line 0 comments Download
M templates/edit.html View 1 chunk +1 line, -1 line 0 comments Download
M templates/publish.html View 1 chunk +1 line, -1 line 0 comments Download
M templates/search.html View 1 chunk +1 line, -1 line 0 comments Download
M templates/settings.html View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 5
jrobbins (corp)
PTAL.
9 years, 9 months ago (2014-07-15 18:46:17 UTC) #1
cmp
lgtm with nit https://codereview.appspot.com/111210043/diff/1/codereview/views.py File codereview/views.py (right): https://codereview.appspot.com/111210043/diff/1/codereview/views.py#newcode258 codereview/views.py:258: 'Only viewable by @chromium and @google ...
9 years, 9 months ago (2014-07-15 18:53:27 UTC) #2
jrobbins (corp)
Fixed nit and committed as https://code.google.com/p/rietveld/source/detail?r=0072644f4ae3e1d4defbba13304179a2f8ccdfc3 On Tue, Jul 15, 2014 at 11:53 AM, <cmp@chromium.org> ...
9 years, 9 months ago (2014-07-15 18:58:49 UTC) #3
iannucci
retro-lgtm
9 years, 9 months ago (2014-07-17 18:34:23 UTC) #4
techtonik
8 years, 6 months ago (2015-10-18 07:16:41 UTC) #5
Message was sent while issue was closed.
This creates issues that people can't find how to make issue private.

https://github.com/rietveld-codereview/rietveld/issues/523
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b