Add read-only banner under navbar
Review Request #8812 — Created March 11, 2017 and submitted
Read-only mode is a setting an admin can enable to prevent writes to
the database. This can be used when the site is under maintenence or
being upgraded. This commit adds a banner under the nav bar when the
site is in read-only mode.
Check that the banner appears in read-only mode
Check that the banner does not appear when not in read-only mode
| Description | From | Last Updated |
|---|---|---|
|
Can you wrap your description at 72 characters? |
 brennie
|
|
|
Please attach a screenshot. |
brennie
|
|
|
Just one newline here? |
 szhang
|
|
|
It looks like these lines should be indented by an addition one space? |
szhang
|
|
|
Can you change these to read-only-... instead? It's nicer than readonly. |
 chipx86
|
|
|
Can you make these colours into constants in defs.less? |
brennie
|
|
|
Can you make these constants in defs.less? |
chipx86
|
|
|
Where does the 20px come from? It'd be nice to have this value in defs.less. |
chipx86
|
|
|
Can you change this to read_only_banner? |
chipx86
|
|
|
This would be much nicer as read_only_banner.html. Although, is there really any reason to split that off into another file? … |
chipx86
|
|
|
Should have the same name as the block. Helps to catch problems as code shuffles around, as Django will do … |
chipx86
|
|
|
Can you rename this to readonly_banner.html ? |
brennie
|
|
|
If this file is included, then this check was already done. No need to duplicate it here. |
chipx86
|
|
|
Instead of "Site" how about "Review Board" |
brennie
|
-
Tool: PEP8 Style Checker Ignored Files: reviewboard/static/rb/css/pages/base.less reviewboard/templates/base/readonlybanner.html reviewboard/templates/base.html Tool: Pyflakes Ignored Files: reviewboard/static/rb/css/pages/base.less reviewboard/templates/base/readonlybanner.html reviewboard/templates/base.html
-
-
-
-
-
-
This would be much nicer as
read_only_banner.html.Although, is there really any reason to split that off into another file? It seems we could just put it inline here. It's faster than doing an include, and is only 3 lines.
-
Should have the same name as the block. Helps to catch problems as code shuffles around, as Django will do a name check and make sure blocks line up.
-
KH
- Change Summary:
-
Wrap description
- Description:
-
~ Read-only mode is a setting an admin can enable to prevent writes to the database. This can be used when the site is under maintenence or being upgraded. This commit adds a banner under the nav bar when the site is in read-only mode.
~ Read-only mode is a setting an admin can enable to prevent writes to
+ the database. This can be used when the site is under maintenence or + being upgraded. This commit adds a banner under the nav bar when the + site is in read-only mode. - Added Files:
-
Tool: Pyflakes Ignored Files: reviewboard/static/rb/css/pages/base.less reviewboard/static/rb/css/defs.less reviewboard/templates/base.html Tool: PEP8 Style Checker Ignored Files: reviewboard/static/rb/css/pages/base.less reviewboard/static/rb/css/defs.less reviewboard/templates/base.html
Tool: Pyflakes Ignored Files: reviewboard/static/rb/css/pages/base.less reviewboard/templates/base/readonlybanner.html reviewboard/templates/base.html Tool: PEP8 Style Checker Ignored Files: reviewboard/static/rb/css/pages/base.less reviewboard/templates/base/readonlybanner.html reviewboard/templates/base.html