Skip to content

Adding some font properties to whitelist for css.#8141

Merged
dereksmart merged 1 commit intomasterfrom
csstidy-add-rules
Nov 14, 2017
Merged

Adding some font properties to whitelist for css.#8141
dereksmart merged 1 commit intomasterfrom
csstidy-add-rules

Conversation

@georgestephanis
Copy link
Copy Markdown
Contributor

Sourced list from https://developer.mozilla.org/en-US/docs/Web/CSS/font following a request from a user.

cc: @brezocordero

@georgestephanis georgestephanis added [Feature] Custom CSS Enhancement Changes to an existing feature — removing, adding, or changing parts of it labels Nov 9, 2017
@jeherve jeherve added the [Status] Needs Review This PR is ready for review. label Nov 10, 2017
Copy link
Copy Markdown
Contributor

@zinigor zinigor left a comment

Choose a reason for hiding this comment

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

Works nice, except for entries that are not yet supported by CodeMirror, like line-height-step which is not yet in any standards. LGTM, let's support draft properties to stay ahead of the curve.

@zinigor zinigor added [Status] Ready to Merge Go ahead, you can push that green button! and removed [Status] Needs Review This PR is ready for review. labels Nov 13, 2017
@dereksmart dereksmart merged commit 2a35563 into master Nov 14, 2017
@dereksmart dereksmart removed the [Status] Ready to Merge Go ahead, you can push that green button! label Nov 14, 2017
@dereksmart dereksmart deleted the csstidy-add-rules branch November 14, 2017 19:34
@georgestephanis
Copy link
Copy Markdown
Contributor Author

georgestephanis commented Nov 15, 2017

Merged upstream to wpcom in r165523-wpcom

@oskosk oskosk added this to the 5.6 milestone Nov 22, 2017
jeherve added a commit that referenced this pull request Nov 24, 2017
oskosk pushed a commit that referenced this pull request Nov 28, 2017
* Changelog 5.6: create base for changelog.

* Update changelog with 5.5.1 info.

* Changelog: add #7930 and #8238

* Changelog: add #8076

* Changelog: add #8100

* Changelog: add #8117

* Changelog: add #8141

* Changelog: add #8143

* Changelog: add #8147

* Changelog: add #8149

* Changelog: add #8153

* Changelog: add #8173

* Changelog: add #8184

* Changelog: add #8196

* Changelog: add #8199

* Changelog: add #8093

* Changelog: add #8171

* Changelog: add #8182

* Changelog: add #8202, #8222

* Changelog: add #8228

* Changelog: add #8240

* Changelog: add #8251

* remove AL card change
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement Changes to an existing feature — removing, adding, or changing parts of it [Feature] Custom CSS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants