DNNCommunity / Dnn.CommunityForums

Open-source forums module for DNN Platform. This is a fork and continuance of the Active Forums module.
https://dnncommunity.org
Other
13 stars 21 forks source link

BUG: Alignment issues on `Active Topics` filtered list #948

Closed david-poindexter closed 4 months ago

david-poindexter commented 4 months ago

Describe the bug

When viewing Active Topics filtered list, there are content alignment issues.

Software Versions

To Reproduce

Steps to reproduce the behavior:

  1. Filter by Active Topics
  2. See alignment issues (screenshot below)

Expected behavior

Alignment should be consistent with other lists.

Screenshots

image

Error log

n/a

Additional context

n/a

Timo-Breumelhof commented 4 months ago

@johnhenley These are not form a template currently right? The main issue here is that we generate a tables in a table and not "connected columns"

I'll just put in some CSS for the width and in v9 I'll re-do a template?

BTW, I also noticed there are no headings for the table #949

Timo-Breumelhof commented 4 months ago

Note to self, this will make the table render better but there's stil the issue of the columns not being part of one table.

/* Search results table, used for search and filtered views of topics */
/* -- Temporary Solution until version 9 -- */

.dnn-community-forums .af-search-results .af-colstats {
    width: 3em;
}

.dnn-community-forums .af-search-results  .af-lastpost{
    width: 15em;
}
johnhenley commented 4 months ago

@david-poindexter @Timo-Breumelhof this is duplicate of #656 I think.

Expecting this view to get a makeover in v8.2 or v9 as we continue to move towards better html practices and moving from hard-coded to templates and resource strings for small snippets.

I did a workaround on my site by adding this to the /themes/community-default/custom/theme.css (I LOVE that feature for layering in the custom CSS šŸ˜Š) This kinda fixes it by adding this, but not really sure what Iā€™m doing šŸ˜Š .afgrid {table-layout: fixed;}

Timo-Breumelhof commented 4 months ago

@johnhenley try the CSS above ;-)

david-poindexter commented 4 months ago

@david-poindexter @Timo-Breumelhof this is duplicate of #656 I think.

Expecting this view to get a makeover in v8.2 or v9 as we continue to move towards better html practices and moving from hard-coded to templates and resource strings for small snippets.

I did a workaround on my site by adding this to the /themes/community-default/custom/theme.css (I LOVE that feature for layering in the custom CSS šŸ˜Š) This kinda fixes it by adding this, but not really sure what Iā€™m doing šŸ˜Š .afgrid {table-layout: fixed;}

My apologies for the duplicate - I somehow missed that. šŸ™‚

Timo-Breumelhof commented 4 months ago

@david-poindexter no problem, thank you for taking the time to log these issues. :-)