Skip to content

Add missing styles to cytoscape css.Node - #34705

Merged
armanio123 merged 1 commit into
masterfrom
cytoscape-add-missing-css-node-props
Apr 15, 2019
Merged

armanio123 merged 1 commit into
masterfrom
cytoscape-add-missing-css-node-props

Conversation

@weswigham

Copy link
Copy Markdown
Contributor

Found while working on microsoft/TypeScript#30853 - the tests use "content", "text-valign" and "text-halign" css properties - these were missing from the Node interface.

@typescript-bot

typescript-bot commented Apr 12, 2019 •

Copy link
Copy Markdown
Contributor

@weswigham Thank you for submitting this PR!

🔔 @phreed @wy193777 @ypconstante @janniclas @cerberuser - please review this PR in the next few days. Be sure to explicitly select Approve or Request Changes in the GitHub UI so I know what's going on.

If no reviewer appears after a week, a DefinitelyTyped maintainer will review the PR instead.

@typescript-bot typescript-bot added Owner Approved A listed owner of this package signed off on the pull request. Merge:Express and removed Awaiting reviewer feedback labels Apr 15, 2019
@typescript-bot

Copy link
Copy Markdown
Contributor

A definition owner has approved this PR ⭐️. A maintainer will merge this PR shortly. If it shouldn't be merged yet, please leave a comment saying so and we'll wait. Thank you for your contribution to DefinitelyTyped!

@armanio123
armanio123 merged commit ea75fef into master Apr 15, 2019
@typescript-bot

Copy link
Copy Markdown
Contributor

I just published @types/[email protected] to npm.

@weswigham
weswigham deleted the cytoscape-add-missing-css-node-props branch April 15, 2019 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Owner Approved A listed owner of this package signed off on the pull request.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants