Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: use custom element constructors instead of tag names #354

Closed
wants to merge 14 commits into from

Conversation

mandurisaimounika
Copy link
Contributor

@mandurisaimounika mandurisaimounika commented Nov 4, 2021

Closes #350

Copy link
Contributor

@ca-d ca-d left a comment

Choose a reason for hiding this comment

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

We usually try to separate internal (end in .js) from external (no .js at the end) dependencies' import statements by an empty line and try to put foundation.js first among internal dependencies.

LGTM otherwise

Copy link
Collaborator

@JakobVogelsang JakobVogelsang left a comment

Choose a reason for hiding this comment

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

LGTM

@JakobVogelsang
Copy link
Collaborator

@ca-d I would suggest doing the rearrangement of all import statements at the end of this refactor. There are just too many inconsistencies at the moment and makes it harder to review if done right now.

@ca-d
Copy link
Contributor

ca-d commented Nov 8, 2021

@ca-d I would suggest doing the rearrangement of all import statements at the end of this refactor.

Sounds good.

@ca-d ca-d changed the title refactor(all-files): change the html imports refactor: use custom element constructors instead of tag names Nov 12, 2021
JakobVogelsang and others added 8 commits November 15, 2021 12:19
* refactor: scope editor-container

* refactor: scope zeroline-pane

* refactor: scope ied-editor
* refactor(all-files): change the mwc-dialog tags to Dialog class

* refactor: declare Dailog class based on DailogBase

* test: update snapshots

Co-authored-by: Sai Mounika Manduri <[email protected]>
Co-authored-by: Sai Mounika Manduri <[email protected]>
…ur-color (#380)

* refactor(all-files): change the mwc-select, mwc-checkbox to class names

* fix(foundation): precise checkValidity

Co-authored-by: Sai Mounika Manduri <[email protected]>
…oggle (#382)

* refactor: scope custom element mwc-icon-button

* refactor: scope costom element mwc-icon-button-toggle
* refactor: scope custom elements mwc-icon

* refactor: scope custom element mwc-menu

* refactor: scope custom element mwc-switch
@ca-d
Copy link
Contributor

ca-d commented Nov 24, 2021

Sadly, we only noticed with the second to last component migrated (tab) that there are hard internal dependencies between mwc-tab and mwc-tab-bar which forbid us from registering our own tag names for mwc elements at the moment.

We may think about raising an issue with the material-web team about future support for Scoped Custom Element Registry .

Thank you for your painstaking work @mandurisaimounika , your changelog was invaluable for us in preparing #395 .

@ca-d ca-d closed this Nov 24, 2021
@JakobVogelsang JakobVogelsang deleted the refactor-html-references branch December 15, 2021 19:23
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.

Scope elements tag names in open-scd
3 participants