Bug Tracker

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#9881 closed enhancement (wontfix)

Capitalize error message

Reported by: anonymous Owned by:
Priority: low Milestone: None
Component: attributes Version: 1.6.2
Keywords: Cc:
Blocked by: Blocking:

Description

I'm sorry for nitpicking, but since all but one error messages have their first letter capitalized, it might be a good idea to capitalize the following error message too.

type property can't be changed

inside jQuery.attrHooks.type.set.

Change History (6)

comment:1 Changed 8 years ago by timmywil

Component: unfiledattributes
Priority: undecidedlow
Resolution: wontfix
Status: newclosed

I think it is a logical choice to leave type lowercase because it is an attribute name.

comment:2 Changed 8 years ago by anonymous

In the error message it is a property.

comment:3 Changed 8 years ago by Rick Waldron

In the error message it is a property.

In which case timmywil's argument still stands

comment:4 Changed 8 years ago by ajpiano

We could use this as an excuse to improve the exception and make it clear to users, perhaps...

"The 'type' property of an input element can't be changed"

(ps I meant to post this two hours ago, forgot to press submit)

comment:5 Changed 8 years ago by timmywil

It's both actually. I'm ok with making the error message clearer. Seems pretty clear to me tho.

comment:6 Changed 8 years ago by Rick Waldron

@timmywil I'm not ok with changing it to be _wrong_, the message should be reworded, if anything

Note: See TracTickets for help on using tickets.