Opened 11 years ago

Closed 11 years ago

#20975 closed Cleanup/optimization (fixed)

Fix incorrect markup of committers list (and possibly other docs lists)

Reported by: Daniele Procida Owned by: Daniel Boeve
Component: Documentation Version: dev
Severity: Normal Keywords: afraid-to-commit
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

The list of Django committers - https://docs.djangoproject.com/en/dev/internals/committers/ - should be three HTML definition lists. In fact, it's dozens of definition lists, of the form:

<dl>
    <dt>Name</dt>
    <dd>Description</dd>
</dl>
<dl>
    <dt>Name</dt>
    <dd>Description</dd>
</dl>
[... and so on]

This is because it is marked up incorrectly.

Incorrect:

Marc Tamlyn
    Marc currently works at `Incuna Ltd`_, [...]

.. _Incuna Ltd: http://incuna.com/

Correct:

Marc Tamlyn
    Marc currently works at `Incuna Ltd`_, [...]

    .. _Incuna Ltd: http://incuna.com/

The incorrect form places the reference for the links in the previous block at the start of the line, rather than indented to the same column as the block. This causes a new definition list to be started the next time the markup for a definition term (i.e. <dt>Name</dt>) is encountered.

It's quite possible that the same problem occurs elsewhere in the documents, so it would be worth checking.

I would like to reserve this ticket for first-time committers who take part in the Don't be afraid to commit tutorial and the sprints at DjangoCon US 2013 in September.

However, if you want to tackle this ticket before then, please don't hesitate. Feel free to re-assign it to yourself and do whatever you like to it.

Change History (6)

comment:1 by Claude Paroz, 11 years ago

Triage Stage: UnreviewedAccepted
Type: UncategorizedCleanup/optimization

comment:2 by Daniele Procida, 11 years ago

Owner: changed from nobody to Daniele Procida
Status: newassigned

comment:3 by Daniele Procida, 11 years ago

Keywords: afraid-to-commit added

comment:4 by anonymous, 11 years ago

Owner: changed from Daniele Procida to anonymous

comment:5 by Daniel Boeve, 11 years ago

Owner: changed from anonymous to Daniel Boeve

comment:6 by Tim Graham <timograham@…>, 11 years ago

Resolution: fixed
Status: assignedclosed

In b69d1eac641433dba3d6e505261f9ad7db384bd7:

Fixed #20975 -- Fixed identation in docs/internals/committers.txt

Thanks EvilDMP for the report.

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