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.

Sign up for GitHub

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

Jump to bottom

ENH: Community Default Theme incorrect "dcf-topic-content" class #732

Closed
Timo-Breumelhof opened this issue Apr 5, 2024 · 0 comments · Fixed by #733
Closed

ENH: Community Default Theme incorrect "dcf-topic-content" class #732

Timo-Breumelhof opened this issue Apr 5, 2024 · 0 comments · Fixed by #733
Assignees
@Timo-Breumelhof
Labels
enhancement New feature or request themes Dnn Community Forums Themes
Milestone
08.01.00

Comments

@Timo-Breumelhof
Copy link
Contributor

Is your feature request related to a problem?

Please describe.
The theme uses the same class (dcf-topic-content) on two different levels.
The top one wraps the items, the bottom one the item's content

Describe the solution you'd like

Change the first one to a more logical class name: dcf-topic-wrap

image
@Timo-Breumelhof Timo-Breumelhof added the enhancement New feature or request label Apr 5, 2024
@Timo-Breumelhof Timo-Breumelhof added this to the 08.01.00 milestone Apr 5, 2024
@Timo-Breumelhof Timo-Breumelhof self-assigned this Apr 5, 2024
@Timo-Breumelhof Timo-Breumelhof added the themes Dnn Community Forums Themes label Apr 5, 2024
Timo-Breumelhof pushed a commit to Timo-Breumelhof/Dnn.CommunityForums that referenced this issue Apr 5, 2024
Change dcf-topic-content to dcf-topic-wrap DNNCommunity#732
320e6df
@Timo-Breumelhof Timo-Breumelhof mentioned this issue Apr 5, 2024
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Timo-Breumelhof Timo-Breumelhof

Labels
enhancement New feature or request themes Dnn Community Forums Themes
Projects
None yet
Milestone
08.01.00
Development

Successfully merging a pull request may close this issue.

Feature/732 default theme dcf topic content class Timo-Breumelhof/Dnn.CommunityForums
1 participant
@Timo-Breumelhof