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

Modal in 2.0-rc.0 again doesn't add page class name to ion-page anymore #8556

Closed
michael-brade opened this issue Oct 7, 2016 · 3 comments
Closed
Assignees

Comments

@michael-brade
Copy link

Hi, I have already written a comment in #7000, but that is closed already. The code that fixed #7000 was deleted again on Jun 28 with this commit 215c6d8#diff-7b81c66d6f3e9c4b82c40ee5ecd30e50 (in fact, the whole loadComponent function was deleted). So now with Ionic 2 RC0 we are back to no class name in a modal dialog. Can you please add it again?

@luchillo17
Copy link
Contributor

Soo as i understand @michael-brade it's a regression?

@brandyscarney
Copy link
Member

We are no longer dynamically adding class names because this caused issues with the latest build process. Instead, you should add a selector to the component and then target the selector as an element. For example in the component:

@Component({
  selector: 'modal-about',
  templateUrl: 'about.html'
})

then in the css:

modal-about {
  // any styles for the component
}

I've updated the changelog to include some more info on this, but please let me know if there's more we can do to make this known: https://github.com/driftyco/ionic/blob/master/CHANGELOG.md#200-rc0-2016-09-28

Thanks!

@michael-brade
Copy link
Author

Thanks, great stuff! That worked!

@ionitron-bot ionitron-bot bot locked and limited conversation to collaborators Sep 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants