Bug Tracker

Modify ↓

Ticket #9881 (closed enhancement: wontfix)

Opened 22 months ago

Last modified 22 months ago

Capitalize error message

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

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

comment:1 Changed 22 months ago by timmywil

  • Priority changed from undecided to low
  • Resolution set to wontfix
  • Status changed from new to closed
  • Component changed from unfiled to attributes

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

comment:2 Changed 22 months ago by anonymous

In the error message it is a property.

comment:3 Changed 22 months ago by rwaldron

In the error message it is a property.

In which case timmywil's argument still stands

comment:4 Changed 22 months 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 22 months 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 22 months ago by rwaldron

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

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View ↑

Add a comment

Modify Ticket

Action
as closed
Author


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

Attachments ↑
 
Note: See TracTickets for help on using tickets.