Vimux / Mainroad

Responsive, simple, clean and content-focused Hugo theme based on the MH Magazine lite WordPress theme
https://mainroad-demo.netlify.app/
GNU General Public License v2.0
909 stars 459 forks source link

Add 'Related Content' section #193

Open comnam90 opened 4 years ago

comnam90 commented 4 years ago

Are there any plans to add a related content section to the single page posts? It'd be nice to have just above the comments section using the built in hugo related content functionality.

Vimux commented 4 years ago

I have no plans, but PRs are always open to everyone.

gooseyman commented 4 years ago

@Vimux I would like to work on this - just read the contribution guidelines, ok to start on this?

Vimux commented 4 years ago

@gooseyman,

OK, but I would like to see design mockups or ideas, to read in a few words what exactly you want to do with this and how.

gooseyman commented 4 years ago

@Vimux I'm proposing that this:

From the default pages, "Getting Started with Hugo" and "(Hu)go Template Primer" are related by the development tag. The image below shows and example of the Related Posts widget in the sidebar.

image

Vimux commented 4 years ago

should be an additional sidebar widget

It should be section. The original title and description state it clear:

https://github.com/Vimux/Mainroad/issues/193#issue-602625024

Are there any plans to add a related content SECTION to the single page posts? It'd be nice to have JUST ABOVE THE COMMENTS section using the built in hugo related content functionality.

By default, Mainroad sidebar contains widgets directly not connected with the current page content. So, no TOC, no comments, no related content widgets inside sidebar.


default to using "tags" as the basis for relating posts

I do not think that this should be the responsibility of the theme.

comnam90 commented 4 years ago

I mean I'm not opposed to the idea of it being a side bar instead. I've just found it more common to have it as a section at the end of a post

gooseyman commented 4 years ago

Understood on #193 (comment) indicating section vs. widget. My proposal was for the widget nonetheless. However, good context on the Mainroad widgets being non-article specific. I agree that it should be a section on the page.

The Wordpress version of theme handles this like so: image https://mhthemes.com/demo/?theme=magazine

I do not think that this should be the responsibility of the theme.

I'm not following what you mean. The relation needs to leverage some variables in order to run - what did you have in mind if not using article tags?

Vimux commented 4 years ago

My quick thoughts about this section:


The relation needs to leverage some variables in order to run...

I mean, we should use standard configuration by default. Something like this: .Site.RegularPages.Related .

REF: https://gohugo.io/content-management/related/#list-related-content https://gohugo.io/content-management/related/#configure-related-content

metbril commented 2 years ago

This issue is little over 2 years old. Any progress or update on this?

Vimux commented 2 years ago

Nothing has changed since then.

  1. I don't plan to work on it. See https://github.com/Vimux/Mainroad/issues/193#issuecomment-616162932
  2. I have not seen any mockups of how it should look&feel nor draft solutions trying to solve the problem (and not get around it or just copying from other themes)
  3. I don't see this feature being popular over these two years.