Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

fixes #2625: don't auto append semicolon after selecting a property value #2675

Merged
merged 1 commit into from
Jan 25, 2013

Conversation

dangoor
Copy link
Contributor

@dangoor dangoor commented Jan 25, 2013

Fix for #2625 with a minor change to CSS code hinting so that semicolons are not added when you select a property value.

@ghost ghost assigned RaymondLim Jan 25, 2013
@@ -175,12 +175,11 @@ define(function (require, exports, module) {
expectCursorAt({ line: 7, ch: 8 });
});

it("should insert semicolon followed by newline after prop-value selected", function () {
// insert semicolon after previous rule to avoid incorrect tokenizing
it("should not insert semicolon after prop-value selected", function () {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice that you remove the incorrect part of the description!

@RaymondLim
Copy link
Contributor

Looks good. Merging.

RaymondLim added a commit that referenced this pull request Jan 25, 2013
fixes #2625: don't auto append semicolon after selecting a property value
@RaymondLim RaymondLim merged commit 52a17aa into master Jan 25, 2013
@RaymondLim RaymondLim deleted the dangoor/css-no-semicolon branch January 25, 2013 19:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants