Closed
Bug 208561
Opened 21 years ago
Closed 21 years ago
simple grammar error in xslt failure message
Categories
(Core :: XSLT, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: e_mayilme, Assigned: peterv)
Details
Attachments
(1 file)
2.25 KB,
patch
|
axel
:
review+
peterv
:
superreview+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.4) Gecko/20030529 Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.4) Gecko/20030529 "Parsing a XSLT stylesheet failed." correctly should be writtten as "Parsing an XSLT stylesheet failed." (x is pronounced with a beginning vowel sound--an is used before all words with a beginning bowel sound) Reproducible: Always Steps to Reproduce: 1. 2. 3.
Reporter | ||
Comment 1•21 years ago
|
||
oops: an is used before all words with a beginning _vowel_ sound.
Comment 2•21 years ago
|
||
Grammatical errors are Severity: trivial. Reporter: Can you provide a public test case?
Severity: normal → trivial
Reporter | ||
Comment 3•21 years ago
|
||
try this page: http://www.providenceproject.com/fmxml/fmpro.xml I don't know if the xml is right or not, but it does show the grammatical mistake in the error message (Mozilla 1.4 rc1 on MacOSX). Perhaps grep -r 'a XSLT' in the Mozilla tree would find the source file, and changing it to "an XSLT" would then be easy.
Reporter | ||
Comment 4•21 years ago
|
||
Here are patches to a few files with the spelling mistake: I checked out the whole CVS to do this, so pls don't ignore. The patch is for these files: mozilla/extensions/transformiix/resources/buster/ReadMe mozilla/extensions/transformiix/resources/xslt.properties Those were the first I found when grepping, so I don't know if there are more. nb: I could not find (quickly) a guideline for the correct patch format, so I used -pru. Should someone add a bug to have this be mentioned directly in the first Bugzilla fact sheet? (Forget it: I just found it, "cvs diff -u > mypatch.diff," although it indirectly assumes one is using CVS.)
Updated•21 years ago
|
Attachment #126262 -
Flags: review+
Assignee | ||
Comment 5•21 years ago
|
||
Comment on attachment 126262 [details] [diff] [review] a patch for the grammar error, i.e, "a xslt" / "a xul" >+The buster is an XUL interface to the conformance tests shipped as part of XUL is pronounced "zool" so I am not sure about this change.
Attachment #126262 -
Flags: superreview+
Comment 6•21 years ago
|
||
Hixie it's "a XUL document" with "an XBL binding" Hixie i say that as the editor of both those specs :-) Hixie (well, one of the editors) checked in, thanx for the fix.
Status: UNCONFIRMED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•