This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 104060 - CSS 2.0 language constructs marked as errors
Summary: CSS 2.0 language constructs marked as errors
Status: VERIFIED FIXED
Alias: None
Product: web
Classification: Unclassified
Component: CSS Visual Tools (show other bugs)
Version: 6.x
Hardware: All Linux
: P3 blocker (vote)
Assignee: Marek Fukala
URL:
Keywords:
: 111405 (view as bug list)
Depends on:
Blocks:
 
Reported: 2007-05-16 12:41 UTC by Jindrich Sedek
Modified: 2008-04-28 13:37 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jindrich Sedek 2007-05-16 12:41:47 UTC
This issue covers CSS 2.0 specification issues.
http://www.w3.org/TR/CSS2/

====================================
1)
CSS editor should not show syntax error for correct selectors constructs:
  E[foo="warning"]{}
  E[foo|="warning"]{}
  E[foo~="warning"]{}

  http://www.w3.org/TR/CSS2/selector.html



=========================================
2)
It's valid to cut strings for more lines using escaping (\)
 - it breaks coloring now
zn {
   font-family: "Times \
   New Roman",Times,serif;
 }
this is the same as:
zn {
   font-family: "Times New Roman",Times,serif;
 }


==================================================
3)
http://www.w3.org/TR/CSS2/grammar.html#tokenizer-diffs - D3, last point

-------citation:---------
In CSS1, a class name could start with a digit (".55ft"), unless it was a
dimension (".55in"). In CSS2, such classes are parsed as unknown dimensions (to
allow for future additions of new units). To make ".55ft" a valid class, CSS2
requires the first digit to be escaped (".\55ft")
--------------------------
There should be no syntax error for 
.\55ft{} neither for .55ft{}
and maybe a warning of invalidity in CSS2 for
.55ft{}

P.S.: css-validator complains the same problem for ids:
#55ft{}
Comment 1 Jindrich Sedek 2007-05-17 13:12:05 UTC
4)
missing pseudo classes in style rule editor:
:first-child, :focus, :lang,
psedo-elements:
:first-line, :first-letter, :before, :after

http://www.w3.org/TR/REC-CSS2/selector.html#pseudo-elements
Comment 2 Marek Fukala 2007-06-19 11:53:59 UTC
reproducible, I'll try to address these in 6.0
Comment 3 Jindrich Sedek 2007-07-31 08:52:14 UTC
*** Issue 111405 has been marked as a duplicate of this issue. ***
Comment 4 Petr Jiricka 2007-08-02 12:08:32 UTC
From the summary this does not sound like a bug. What is the real bug? Please change the summary so it's apparent what
the user problem is.

Comment 5 Marek Fukala 2007-08-02 21:50:07 UTC
The issue is about some CSS 2.0 language constructs not currently supported by CSS editor. I agree that it is not a
critical issue, but we should not support just CSS 1.0, that would be very bad.

So I am lowering the priority to P3 for now, so we do not to tease the dashboards much, but the issue really should be
fixed for 6.0 - which was the reason for being P2 - not to be lost in the bunch of P3s. It's a typical P2,5 :-)

To be more specific, the grammar in NBS file needs to be changed so it can parse CSS 2.0 and doesn't mark valid
constructs like bugs.
Comment 6 Marek Fukala 2007-08-21 11:58:00 UTC
#1, #2 fixed

as for #3 - fixed, the enhancement for marking the deprecated css1.0 syntax file as a separate issue if you want.

#4 is and enhancement for the new css rule wizard, please file separatedly, thanks.

Checking in CSS.nbs;
/cvs/languages/css/src/org/netbeans/modules/languages/css/CSS.nbs,v  <--  CSS.nbs
new revision: 1.19; previous revision: 1.18
done
Comment 7 Jindrich Sedek 2007-09-21 15:59:05 UTC
verified.
NetBeans IDE Dev (Build 200709200000)