Opened 9 years ago
Closed 9 years ago
#25667 closed Bug (fixed)
Misspelling in docutils setting override
Reported by: | Ville Skyttä | Owned by: | nobody |
---|---|---|---|
Component: | contrib.admindocs | Version: | dev |
Severity: | Normal | Keywords: | |
Cc: | Triage Stage: | Accepted | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
Should be initial_header_level
, not inital_header_level
.
Change History (7)
comment:1 by , 9 years ago
comment:2 by , 9 years ago
Has patch: | set |
---|
Frankly, I don't know the answer to either, I just happened to stumble upon this while checking for general misspellings.
comment:3 by , 9 years ago
Needs tests: | set |
---|---|
Triage Stage: | Unreviewed → Accepted |
I think it makes sense, but we should try to add a test.
comment:4 by , 9 years ago
Needs tests: | unset |
---|
I took a look, and in my opinion initial header level 3 is desirable; h3 looks better than h1 there, and the page already contains a h1 (and seemingly also a h2) and there at least used to be a general recommendation not to have more than one h1 in web pages. PR force-updated with a test case.
comment:5 by , 9 years ago
Patch needs improvement: | set |
---|
Test could use some improvement as noted on the PR.
As this has been broken forever ("Imported Django from private SVN repository") contains the misspelling, I wonder if we need to fix it (or what change fixing it causes)?