Skip to content

add note about semicolon#120

Open
ooker777 wants to merge 1 commit intoHEmile:mainfrom
ooker777:patch-5
Open

add note about semicolon#120
ooker777 wants to merge 1 commit intoHEmile:mainfrom
ooker777:patch-5

Conversation

@ooker777
Copy link
Contributor

No description provided.

@HEmile
Copy link
Owner

HEmile commented Feb 22, 2022

I won't merge this as-is, the note is too confusing, and not in the right place. Also, all examples show it with a semicolon, so I'm not sure why it is to be assumed otherwise.

@ooker777
Copy link
Contributor Author

ooker777 commented Feb 22, 2022

well it's just to make sure that it is not assumed thought. Because for proper CSS one can omit the semicolon.

the note is too confusing, and not in the right place.

Do you have any suggestion?

@HEmile
Copy link
Owner

HEmile commented Feb 22, 2022

Do you have any suggestion?

Probably at the end, under limitations, as it's probably a cytoscape css implementation limitation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants