Unit Tests: CSSUtils.getInfoAtPos() ignores prefixes

[oslego]

Is there any reason to ignore prefixes in CSS properties?

Any prefix-like notation is dropped (-webkit- or -xxx-) and the remainder is served as PROP_NAME. This yields false-positives for things like -xxx-transform: value.

Attached test to highlight the issue.

This is for #6861

[redmunds]

@RaymondLim Do you know why CSSUtils ignores prefixes?

[redmunds]

@oslego Does this cause a bug with the Shapes editor, or were you just reporting this as a Brackets bug?

[TomMalbran]

I believe the problem is mainly with CodeMirror, since it doesn't add the prefixes to the properties. Is easy to see with the highlighting. The prefixes are always black, and the properties not.

[RaymondLim]

@TomMalbran You're right! But we can also work around in our CSSUtils.js to collect the prefix and then pass that info to the hint providers.

[oslego]

It does impact the shapes editor, but it's not a critical bug. I wanted to
restrict the functionality only to non-prefixed properties for
future-proofing.

Reporting this as a generic bug, which may influence future extensions.

On Monday, February 10, 2014, Randy Edmunds notifications@github.com
wrote:

@oslego https://github.com/oslego Does this cause a bug with the Shapes
editor, or were you just reporting this as a Brackets bug?

Reply to this email directly or view it on GitHubhttps://github.com//pull/6824#issuecomment-34667113
.

[RaymondLim]

Reassign it to me.

[RaymondLim]

Looks good. Merging.