Skip to content
This repository has been archived by the owner on Jun 21, 2022. It is now read-only.

refactor(GroupData): Fold “CSSOM View” into “CSS Object Model” #1154

Merged
merged 1 commit into from May 16, 2019

Conversation

ExE-Boss
Copy link
Contributor

@ExE-Boss ExE-Boss commented May 15, 2019

See this proposal on Discourse for details.

I’ve also added interfaces that were added to the specs since CSSOM Group Data was last updated. (Moved to: #1156)


P.S.: CaretPosition is actually defined in CSSOM View and not in CSSOM.

review?(@wbamberg)

@wbamberg
Copy link

Thanks for the PR!

I’ve also added interfaces that were added to the specs since CSSOM Group Data was last updated.

Please, don't do this. If the PR is to fold CSSOM View into CSSOM, let it just do that.

@ExE-Boss ExE-Boss force-pushed the macros/group-data/fold-cssom-view branch from 4462bbf to 3f82613 Compare May 16, 2019 00:23
@ExE-Boss ExE-Boss force-pushed the macros/group-data/fold-cssom-view branch 2 times, most recently from 4462bbf to 9fc5cc3 Compare May 16, 2019 00:32
@ExE-Boss
Copy link
Contributor Author

review?(@wbamberg): I’ve now extracted the added interfaces into #1156.

Copy link

@wbamberg wbamberg left a comment

Choose a reason for hiding this comment

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

Thanks @ExE-Boss , this looks great.

@wbamberg wbamberg merged commit bd41307 into mdn:master May 16, 2019
@ExE-Boss ExE-Boss deleted the macros/group-data/fold-cssom-view branch May 16, 2019 01:27
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.

None yet

2 participants