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

Implemented core support for portal styles #5851

Merged
merged 17 commits into from
Dec 17, 2023

Conversation

valadas
Copy link
Contributor

@valadas valadas commented Oct 27, 2023

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.

@valadas valadas added this to the 10.0.0 milestone Oct 27, 2023
DNN Platform/Library/Entities/Portals/StyleColor.cs Outdated Show resolved Hide resolved
DNN Platform/Website/Default.aspx.cs Outdated Show resolved Hide resolved
DNN Platform/Website/Default.aspx.cs Outdated Show resolved Hide resolved
Copy link
Contributor

@GerardSmit GerardSmit left a 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.

DNN Platform/Library/Entities/Portals/StyleColor.cs Outdated Show resolved Hide resolved
DNN Platform/Library/Entities/Portals/StyleColor.cs Outdated Show resolved Hide resolved
DNN Platform/Library/Entities/Portals/StyleColor.cs Outdated Show resolved Hide resolved
valadas and others added 6 commits November 10, 2023 13:42
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..
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.

This is looking good @valadas 🎉

@valadas valadas marked this pull request as ready for review December 9, 2023 07:21
@valadas
Copy link
Contributor Author

valadas commented Dec 9, 2023

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.

valadas added a commit to valadas/Dnn.Platform that referenced this pull request Dec 15, 2023
This is still a work in progress and when dnnsoftware#5851 will be merged, this PR will become way easier to review
@valadas valadas changed the title Started implementation of portal styles Implemented core support for portal styles Dec 16, 2023
@valadas
Copy link
Contributor Author

valadas commented Dec 17, 2023

Awesome, I'll merge this to make #5884 easier to review :)

@valadas valadas merged commit c42e14d into dnnsoftware:release/10.0.0 Dec 17, 2023
3 checks passed
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.

4 participants