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

RESOLVED FIXED

Status

()

Core
CSS Parsing and Computation
--
trivial
RESOLVED FIXED
13 years ago
11 years ago

People

(Reporter: Jesse Ruderman, Assigned: dbaron)

Tracking

Trunk
Points:
---
Bug Flags:
wanted1.9 +

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [patch], URL)

Attachments

(1 attachment)

(Reporter)

Description

13 years ago
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.
As a simple fix, we could put a '.' after "inaccessible property".
Flags: blocking1.9?
Flags: blocking1.9?
Whiteboard: [wanted-1.9]
Status: NEW → ASSIGNED
OS: Mac OS X → All
Hardware: Macintosh → All
Whiteboard: [wanted-1.9] → [wanted-1.9][patch]
Created attachment 252409 [details] [diff] [review]
patch

This also renames two property names whose strings were previously changed, to force retranslation by those locales that didn't already.
Attachment #252409 - Flags: superreview?(bzbarsky)
Attachment #252409 - Flags: review?(bzbarsky)
(Reporter)

Comment 3

12 years ago
What does it mean for a property to be inaccessible?
That you can't actually set it?  Should I say "internal property" instead?
(Reporter)

Comment 5

12 years ago
Ahh.  Yes, that would be better.
In my tree I've changed it to "Cannot specify value for internal property."
Comment on attachment 252409 [details] [diff] [review]
patch

Sounds great, with the new wording
Attachment #252409 - Flags: superreview?(bzbarsky)
Attachment #252409 - Flags: superreview+
Attachment #252409 - Flags: review?(bzbarsky)
Attachment #252409 - Flags: review+
Checked in to trunk.
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
Flags: wanted1.9+
Whiteboard: [wanted-1.9][patch] → [patch]
You need to log in before you can comment on or make changes to this bug.