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

defaultDomElementGetter not using the same id convention as other helper libs #51

Closed
tewsonseeoun-toast opened this issue May 31, 2019 · 2 comments

Comments

@tewsonseeoun-toast
Copy link

single-spa-react has defaultDomElementGetter() for creating a DOM element for an app to mount on. defaultDomElementGetter() uses the app name as the id of the element. https://github.com/CanopyTax/single-spa-react/blob/v2.10.1/src/single-spa-react.js#L160

However, this behaviour is different from other helper libraries. In other helper libraries, the id is single-spa-application:${name}.

There might be some benefits in having a consistent pattern of the default DOM element id?

@joeldenning
Copy link
Member

joeldenning commented May 31, 2019

👍 agreed, let's change single-spa-react to follow the convention in other libs. Doing so will allow https://github.com/CanopyTax/single-spa-inspector to have a default id for overlays.

PR's accepted, or myself or a core team member might be able to get to this in the next day or two.

@joeldenning
Copy link
Member

Released in https://github.com/CanopyTax/single-spa-react/releases/tag/v2.10.2

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

No branches or pull requests

2 participants