315250 - "Attempt to use inaccessible property Error in parsing value for property" is confusing wording

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, defect)

Description

[Jesse Ruderman]

Steps to reproduce:
1. Load a page containing <style>* { margin-left-value: 5; }</style>

Result: "Error: Attempt to use inaccessible property Error in parsing value for property 'margin-left-value'."

Expected: An error message that's easier to understand or at least easier to parse into English.  "Error: Attempt to use inaccessible property 'margin-left-value'." would work.

Comment 1

[Boris Zbarsky [:bzbarsky]]

As a simple fix, we could put a '.' after "inaccessible property".

Comment 2

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Attachment: patch

This also renames two property names whose strings were previously changed, to force retranslation by those locales that didn't already.

Comment 3

[Jesse Ruderman]

What does it mean for a property to be inaccessible?

Comment 4

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

That you can't actually set it?  Should I say "internal property" instead?

Comment 5

[Jesse Ruderman]

Ahh.  Yes, that would be better.

Comment 6

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

In my tree I've changed it to "Cannot specify value for internal property."

Comment 7

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 252409 [details] [diff] [review]
patch

Sounds great, with the new wording

Comment 8

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Checked in to trunk.