-
Notifications
You must be signed in to change notification settings - Fork 748
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
Implemented core support for portal styles #5851
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some small things I saw while looking at this PR. These aren't big issues since the PortalStyles and the CSS are being cached: it's only when the cache doesn't exist these allocations happen.
Started implementation of portal styles. This is incomplete and a work in progress for now...
We may change our minds on these after we get the new default.css in place and the new theme but this should roughly match the current default.css values..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is looking good @valadas 🎉
Ok, I think this is good to merge now, I started work on the actual management module which I would like to submit a different PR for to make it easier to review than a huge PR and also makes this available in DNN for the theme development. |
DNN Platform/DotNetNuke.Abstractions/Portals/IPortalSettings.cs
Outdated
Show resolved
Hide resolved
This is still a work in progress and when dnnsoftware#5851 will be merged, this PR will become way easier to review
Awesome, I'll merge this to make #5884 easier to review :) |
Implementation of portal styles for the core level support.
This is complete but the overall support, in next PRs I'll submit a module to manage those settings and we can then consume those css values in dnn-elements, resource-manger, new theme, etc.