Code

Opened 5 years ago

Closed 4 years ago

Last modified 4 years ago

#11509 closed (fixed)

Incorrect capitalisation of web in Widget documentation

Reported by: thepointer Owned by: dwillis
Component: Documentation Version: master
Severity: Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

http://docs.djangoproject.com/en/dev/ref/forms/widgets/#customizing-widget-instances - last paragraph before Widget.attrs:

"On a real web page, you probably don't want every widget to look the same. You might want a larger input element for the comment, and you might want the 'name' widget to have some special CSS class. To do this, you use the attrs argument when creating the widget:"

Should be:

"On a real Web page, you probably don't want every widget to look the same. You might want a larger input element for the comment, and you might want the 'name' widget to have some special CSS class. To do this, you use the attrs argument when creating the widget:"

I'd suggest doing a case sensitive 'find all' for 'web' through the documentation. This change should be applied to bring the documentation in line with http://docs.djangoproject.com/en/dev/internals/contributing/#commonly-used-terms

Attachments (3)

11509.diff (32.9 KB) - added by dwillis 4 years ago.
11509.2.diff (32.9 KB) - added by timo 4 years ago.
update to apply cleanly to trunk
11509.3.diff (47.2 KB) - added by DrMeers 4 years ago.
More thorough patch

Download all attachments as: .zip

Change History (10)

comment:1 Changed 5 years ago by ubernostrum

  • milestone 1.0.3 deleted
  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset

There's no way on earth that this should be a blocker for 1.0.3.

comment:2 Changed 5 years ago by Alex

  • Triage Stage changed from Unreviewed to Accepted

comment:3 Changed 4 years ago by dwillis

  • Owner changed from nobody to dwillis
  • Status changed from new to assigned

Changed 4 years ago by dwillis

comment:4 Changed 4 years ago by dwillis

  • Has patch set
  • Version changed from 1.0 to SVN

Changed 4 years ago by timo

update to apply cleanly to trunk

comment:5 Changed 4 years ago by timo

  • Triage Stage changed from Accepted to Ready for checkin

Changed 4 years ago by DrMeers

More thorough patch

comment:6 Changed 4 years ago by russellm

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [14069]) Fixed #11509 -- Modified usage of "Web" to match our style guide in various documentation, comments and code. Thanks to timo and Simon Meers for the work on the patch.

comment:7 Changed 4 years ago by russellm

(In [14072]) [1.2.X] Fixed #11509 -- Modified usage of "Web" to match our style guide in various documentation, comments and code. Thanks to timo and Simon Meers for the work on the patch

Backport of r14069 from trunk.

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
as The resolution will be set. Next status will be 'closed'
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.