Opened 5 years ago

Closed 5 years ago

#25667 closed Bug (fixed)

Misspelling in docutils setting override

Reported by: Ville Skyttä Owned by: nobody
Component: contrib.admindocs Version: master
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 Changed 5 years ago by Tim Graham

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)?

comment:2 Changed 5 years ago by Ville Skyttä

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 Changed 5 years ago by Tim Graham

Needs tests: set
Triage Stage: UnreviewedAccepted

I think it makes sense, but we should try to add a test.

comment:4 Changed 5 years ago by Ville Skyttä

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 Changed 5 years ago by Tim Graham

Patch needs improvement: set

Test could use some improvement as noted on the PR.

comment:6 Changed 5 years ago by Ville Skyttä

Patch needs improvement: unset

PR force-updated

comment:7 Changed 5 years ago by Tim Graham <timograham@…>

Resolution: fixed
Status: newclosed

In 4d0f883:

Fixed #25667 -- Fixed admindocs initial_header_level.

Note: See TracTickets for help on using tickets.
Back to Top