Sigil-Ebook / flightcrew

Automatically exported from code.google.com/p/flightcrew
GNU General Public License v3.0
34 stars 11 forks source link

incorreclty raising missing target attributes errors, for a elements. #38

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?

See the attached epub, it was generated using steps 1 and 2 here below :

1. Gather one or more articles with the GrabmyBooks Firefox extension, 
if your article contains links, they are generated with an a tag with a 
target="_blank". 
( this was tested with many different sources, Grabmybooks's behavior to 
generate links stays consistent)

2. Export the epub out of the Grabmybook Firefox extension. 

3. Open it in Sigil and try to validate it, 
all links are flagged with an error :
" attribute 'target' is not declared for element 'a' " 
But all a elements do have a target="_blank". 

-> false positive ?

What is the expected output? What do you see instead?

No errors flagged. 
Unless target="_blank" is not allowed, in which case the validation error 
message should be different or more precise.

What version of the product are you using? On what operating system?

The one integrated in Sigil 0.7.1,  on Win 7 pro 64bit

Original issue reported on code.google.com by david.di...@gmail.com on 9 Apr 2013 at 2:52

Attachments:

GoogleCodeExporter commented 9 years ago
Target is not an allowed attribute for the 'a' element. The error message is a 
standard message for such as case, it will not be changed.

Original comment by john@nachtimwald.com on 9 Apr 2013 at 11:35

GoogleCodeExporter commented 9 years ago
Ok. 
But then it's misleading. 
I thought that it was asking for the target attribute on the a element.
Apparently it's the opposite.
So I'd still suggest to rephrase to : 
" attribute 'target' is not ALLOWED for element 'a' " 

Original comment by david.di...@gmail.com on 10 Apr 2013 at 8:17