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

Fixed an issue where the React common components would interfere with container css class #4109

Merged
merged 1 commit into from
Sep 21, 2020

Conversation

valadas
Copy link
Contributor

@valadas valadas commented Sep 20, 2020

Closes #4107
This scopes the container styles inside the area class for the Scrollbars common component. It looks like this component is only used once in all our modules in the Pages module for the list of pages scroll area. I have done some testing and it looks all like before.

Before:
image

After:
image

… container class

Closes dnnsoftware#4107
This scopes the `container` styles inside the `area` class for the Scrollbars common component. It looks like this component is only used once in all our modules in the Pages module for the list of pages scroll area. I have done some testing and it looks all like before.
@valadas valadas added this to the 9.7.3 milestone Sep 20, 2020
@valadas
Copy link
Contributor Author

valadas commented Sep 21, 2020

Just adding a little more context for reviewers, the .container in question is in the .area div so this keeps working in the component while not affecting bootstrap (and other) themes. And that common component appears to be only used in the Pages module for the page list, I tested it there and no UI changes where noticed.

Copy link
Contributor

@david-poindexter david-poindexter left a comment

Choose a reason for hiding this comment

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

Looks good - thanks @valadas 🎉🎉🎉

@mitchelsellers mitchelsellers merged commit 0db1cd2 into dnnsoftware:develop Sep 21, 2020
@valadas valadas modified the milestones: 9.7.3, 9.8.0 Oct 7, 2020
@valadas valadas deleted the issue4107 branch April 14, 2022 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dnn.React.Common bundle makes containers white
3 participants