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 144204 - incorrect warning for background-possition attribute
Summary: incorrect warning for background-possition attribute
Status: RESOLVED FIXED
Alias: None
Product: web
Classification: Unclassified
Component: CSS Visual Tools (show other bugs)
Version: 6.x
Hardware: Sun All
: P3 blocker with 3 votes (vote)
Assignee: Marek Fukala
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-08-17 15:39 UTC by Jindrich Sedek
Modified: 2012-04-26 07:56 UTC (History)
2 users (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 2008-08-17 15:39:49 UTC
use this code:
-------------------
table {
	background-position:center top;
}
------------------
the line is yellow underlined with a warning, but the code is correct 
P.S. try to switch center, top order to 
------------------
background-position:top center ;
------------------
warning disapears, but possitions are separated by || in CSS values grammer so there should be no influence of the order
Comment 1 Marek Fukala 2008-08-21 15:58:58 UTC
reproducible, added currently failing unit test PropertyModelTest.testBackroundPositionOrder()
Comment 2 David Konecny 2008-11-20 21:44:32 UTC
General comment: adding PropertyModelTest.testBackroundPositionOrder() is good but we should annotate these tests
somehow and exclude them from regular run or something. I run tests before and after any change to verify that I have
not broken anything else. And if a test fails I have no idea to know whether it is by designed or not.
Comment 3 Marek Fukala 2008-11-21 18:53:59 UTC
You are completely right. I just remember it, doable for a limited number of tests. Is there such mechanism Davide? 
Comment 4 David Konecny 2008-11-24 00:06:54 UTC
None I would not of. I thought we could use @RandomelyFails from org.nb.junit but its purpose is different. I would
probably comment out (or "if (true) return") the test until the issue is fixed. And fixing issue should put the test
back. Just an idea. What do you think?
Comment 5 Marek Fukala 2008-11-24 11:18:49 UTC
It looks we could use junit4's @Ignore annotation so the test could look like:

@Test
@Ignore("issue XXXX")
public void jindrasCase() {
   ...
}

The problem seems to be that junit switches to some pre 4 compatibility mode if the test class extends TestCase. In such
case the annotation are not processed. If we do not extend the NbTestCase then some of the used nbjunit functionality
misses.

Comment 6 Marek Fukala 2012-04-26 07:56:58 UTC
already fixed