Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#14835 closed (fixed)

Improper indentation for docs/ref/contrib/admin/index.txt

Reported by: Greg Turner Owned by: Gabriel Hurley
Component: Documentation Version: 1.2
Severity: Keywords: sprintdec2010
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

In docs/ref/contrib/admin/index.txt, the attributes for ModelAdmin aren't indented. The effect in the resulting HTML is that <dt> elements are created with no <dd> elements. Instead, there is an empty <dt>, the enclosing <dl> is terminated and the definition appears in the following <p>. For InlineModelAdmin, the markup is correct, but the styling indents the dt, which I guess looks ugly to some.

Both problems are addressed by modifying djangoproject CSS so that <dt> does not appear indented, and also to fix the markup so that <dt>s are populated. I'll do so once we establish a way to update djangoproject's css (discussion at http://groups.google.com/group/django-developers/browse_thread/thread/2d14c16358069b37).

Change History (5)

comment:1 by Greg Turner, 13 years ago

milestone: 1.3

comment:2 by Gabriel Hurley, 13 years ago

milestone: 1.3
Owner: changed from Greg Turner to Gabriel Hurley
Status: newassigned
Summary: Indents for descriptions of documentation attributesImproper indentation for docs/ref/contrib/admin/index.txt
Triage Stage: UnreviewedAccepted

I've noted it on other tickets before, but this is a reST markup problem, not a CSS problem. Basically, it's a lack of proper indentation in the file such that the <dd> elements aren't created. There are a lot of older chunks of the docs that have this problem.

When the markup is correct the indentation of the elements is also correct. I prefer fixing the markup (the real problem) to applying CSS hacks to correct for bad markup.

I'm changing this ticket to be about fixing the markup in that file, and will commit the fix to close it momentarily.

comment:3 by Gabriel Hurley, 13 years ago

Resolution: fixed
Status: assignedclosed

(In [14830]) Fixed #14835 -- Corrected a lack of indentation in the reST markup for docs/ref/contrib/admin/index.txt and reflowed the text accordingly. Thanks to cogat for the report.

comment:4 by Luke Plant, 13 years ago

In [15620]:

[1.2.X] Fixed #14835 -- Corrected a lack of indentation in the reST markup for docs/ref/contrib/admin/index.txt and reflowed the text accordingly.

Thanks to cogat for the report.

Backport of [14830] from trunk - approximate, because other changes have
made it very hard to do exactly.

comment:5 by Jacob, 13 years ago

milestone: 1.3

Milestone 1.3 deleted

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