• 
      

    Fix the display of the review request permission denied page.

    Review Request #7642 — Created Sept. 18, 2015 and submitted

    Information

    Review Board
    release-2.5.x
    5456033...

    Reviewers

    The template for the review request permission denied page had a bunch of extra
    markup that was breaking the box. This fixes it.

    While I was in here, I noticed that our 403 handler (which should hopefully
    never be used) had some TODO text. I've fixed that up too.

    Looked at the permission denied page and saw that things looked pretty.

    Description From Last Updated

    This is a little repetitive. How about we change this to help be more informative, with something like: {% if …

    chipx86chipx86

    Small typo: "an" should be "and".

    AD adriano

    Mind using blocktrans, so thes can wrap nicely?

    chipx86chipx86

    Either the <h1> and the opening <p> need to be indented one more space, or the </p> needs to be …

    mike_conleymike_conley
    reviewbot
    1. Tool: Pyflakes
      Ignored Files:
          reviewboard/templates/403.html
          reviewboard/templates/reviews/review_request_permission_denied.html
      
      
      
      Tool: PEP8 Style Checker
      Ignored Files:
          reviewboard/templates/403.html
          reviewboard/templates/reviews/review_request_permission_denied.html
      
      
    2. 
        
    chipx86
    1. 
        
    2. reviewboard/templates/403.html (Diff revision 1)
       
       
      Show all issues

      This is a little repetitive. How about we change this to help be more informative, with something like:

      {% if request.user.is_anonymous %}
      <p>This may be because you're not logged in. Try logging in an viewing the page again. If you're still seeing this, talk to your administrator.</p>
      {% else %}
      <p>This usually means your account hasn't been granted access to this page. If you feel this is in error, talk to your administrator.</p>
      {% endif %}
      
    3. 
        
    david
    reviewbot
    1. Tool: Pyflakes
      Ignored Files:
          reviewboard/templates/403.html
          reviewboard/templates/reviews/review_request_permission_denied.html
      
      
      
      Tool: PEP8 Style Checker
      Ignored Files:
          reviewboard/templates/403.html
          reviewboard/templates/reviews/review_request_permission_denied.html
      
      
    2. 
        
    AD
    1. 
        
    2. reviewboard/templates/403.html (Diff revision 2)
       
       
      Show all issues

      Small typo: "an" should be "and".

    3. 
        
    david
    reviewbot
    1. Tool: PEP8 Style Checker
      Ignored Files:
          reviewboard/templates/403.html
          reviewboard/templates/reviews/review_request_permission_denied.html
      
      
      
      Tool: Pyflakes
      Ignored Files:
          reviewboard/templates/403.html
          reviewboard/templates/reviews/review_request_permission_denied.html
      
      
    2. 
        
    chipx86
    1. 
        
    2. reviewboard/templates/403.html (Diff revision 3)
       
       
       
       
      Show all issues

      Mind using blocktrans, so thes can wrap nicely?

    3. 
        
    david
    reviewbot
    1. Tool: PEP8 Style Checker
      Ignored Files:
          reviewboard/templates/403.html
          reviewboard/templates/reviews/review_request_permission_denied.html
      
      
      
      Tool: Pyflakes
      Ignored Files:
          reviewboard/templates/403.html
          reviewboard/templates/reviews/review_request_permission_denied.html
      
      
    2. 
        
    mike_conley
    1. 
        
    2. Show all issues

      Either the <h1> and the opening <p> need to be indented one more space, or the </p> needs to be unindented one. In either case, something's wrong here.

    3. 
        
    david
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-2.5.x (9ee0436)