Checklist options manager view

Review Request #6190 — Created Aug. 1, 2014 and discarded

Information

rb-extension-pack

Reviewers

Created a page for checklist templates options.
Connected .less file styleing through TemplateHook.
Created basic structure for the page.
Added styling for the page.

Open [server]/checklist/manager/
The page gets loaded with the view, and its style.


Description From Last Updated

Can this be changed to checklist-manager to be consistent with your other ids? (hyphen instead of underscore)

PE PeterTran

Overtabbed?

PE PeterTran

Html elements should be in alphabetical order

PE PeterTran

Is this a tab instead of space?

PE PeterTran

Rules should be alphabetical order. I've noticed you use 1500 in several spots. If this ever needs to change, it …

PE PeterTran

Tab instead of space?

PE PeterTran

IDs should also be in alphabetical order. Move this up and slide #options-* below #mytemplates-list?

PE PeterTran

Over tabbed?

PE PeterTran

Tabs instead of spaces?

PE PeterTran

The tabbing is not aligned here.

PE PeterTran

The indentation is not flush.

PE PeterTran

Trailing white space.

PE PeterTran

Extra white space between class and id.

PE PeterTran

Maybe put "Name:" on a new line for consistency with line 34?

PE PeterTran

Col: 25 E127 continuation line over-indented for visual indent

reviewbotreviewbot

Col: 1 E124 closing bracket does not match visual indentation

reviewbotreviewbot
reviewbot
  1. Tool: PEP8 Style Checker
    Processed Files:
        checklist/checklist/extension.py
    
    Ignored Files:
        checklist/checklist/templates/checklist/manager.html
        checklist/checklist/static/css/manager.less
        checklist/checklist/templates/checklist/manager_style.html
    
    
    
    Tool: Pyflakes
    Processed Files:
        checklist/checklist/extension.py
    
    Ignored Files:
        checklist/checklist/templates/checklist/manager.html
        checklist/checklist/static/css/manager.less
        checklist/checklist/templates/checklist/manager_style.html
    
    
  2. 
      
PE
  1. 
      
  2. Show all issues

    Can this be changed to checklist-manager to be consistent with your other ids? (hyphen instead of underscore)

  3. Show all issues

    Overtabbed?

  4. checklist/checklist/static/css/manager.less (Diff revision 1)
     
     
     
     
     
     
     
     
     
     
     
    Show all issues

    Html elements should be in alphabetical order

  5. Show all issues

    Is this a tab instead of space?

  6. checklist/checklist/static/css/manager.less (Diff revision 1)
     
     
     
     
     
     
     
    Show all issues

    Rules should be alphabetical order.

    I've noticed you use 1500 in several spots. If this ever needs to change, it might be convenient to store it in a variable?

  7. Show all issues

    Tab instead of space?

  8. Show all issues

    IDs should also be in alphabetical order. Move this up and slide #options-* below #mytemplates-list?

  9. Show all issues

    Over tabbed?

  10. checklist/checklist/static/css/manager.less (Diff revision 1)
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
    Show all issues

    Tabs instead of spaces?

  11. checklist/checklist/static/css/manager.less (Diff revision 1)
     
     
     
     
     
     
     
    Show all issues

    The tabbing is not aligned here.

  12. checklist/checklist/templates/checklist/manager.html (Diff revision 1)
     
     
     
     
     
     
     
     
     
     
     
    Show all issues

    The indentation is not flush.

  13. Show all issues

    Trailing white space.

  14. Show all issues

    Extra white space between class and id.

  15. Show all issues

    Maybe put "Name:" on a new line for consistency with line 34?

  16. 
      
SA
SA
reviewbot
  1. Tool: PEP8 Style Checker
    Processed Files:
        checklist/checklist/extension.py
        checklist/checklist/urls.py
        checklist/checklist/views.py
    
    Ignored Files:
        checklist/checklist/templates/checklist/manager.html
        checklist/checklist/static/css/manager.less
        checklist/checklist/templates/checklist/manager_style.html
    
    
    
    Tool: Pyflakes
    Processed Files:
        checklist/checklist/extension.py
        checklist/checklist/urls.py
        checklist/checklist/views.py
    
    Ignored Files:
        checklist/checklist/templates/checklist/manager.html
        checklist/checklist/static/css/manager.less
        checklist/checklist/templates/checklist/manager_style.html
    
    
  2. checklist/checklist/urls.py (Diff revision 2)
     
     
    Show all issues
    Col: 25
     E127 continuation line over-indented for visual indent
    
  3. checklist/checklist/urls.py (Diff revision 2)
     
     
    Show all issues
    Col: 1
     E124 closing bracket does not match visual indentation
    
  4. 
      
SA
Review request changed
Status:
Discarded