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

Prev next post links on single post page #180

Merged
merged 11 commits into from
Jan 27, 2021
Merged

Conversation

adityatelange
Copy link
Owner

@adityatelange adityatelange commented Jan 16, 2021

  • In Site config: Params.ShowPostNavLinks: true

@adityatelange adityatelange linked an issue Jan 16, 2021 that may be closed by this pull request
@adityatelange
Copy link
Owner Author

adityatelange commented Jan 16, 2021

Demo

image

image

@kdkasad
Copy link
Contributor

kdkasad commented Jan 16, 2021

Looks great!

@adityatelange
Copy link
Owner Author

Looks great!

I still feel the position isn't great, and I am not having any other ideas to fix this.

@kdkasad
Copy link
Contributor

kdkasad commented Jan 17, 2021

Looks great!

I still feel the position isn't great, and I am not having any other ideas to fix this.

I wonder if for the mobile layout, it might be better to omit the titles of the next/previous posts and instead use a button like the Next Page button from the homepage. That way they'd fit side-by-side.

@adityatelange
Copy link
Owner Author

adityatelange commented Jan 17, 2021

Looks great!

I still feel the position isn't great, and I am not having any other ideas to fix this.

I wonder if for the mobile layout, it might be better to omit the titles of the next/previous posts and instead use a button like the Next Page button from the homepage. That way they'd fit side-by-side.

Yes you are correct, I have tried it but it doesn't look good at all with long text titles and also with emoji's.
(On the Home Page the buttons aren't user editable, making it smaller)

oh I got you now, by title you mean the Page title and not "Next Page" one.
IMO it is important as far as UX is concerned, if I am clicking on a link without knowing what's the content is not good.

@muhammadhanif
Copy link
Contributor

Demo

image

image

In my opinion, the prev-next post links would be nice if it is placed below of share buttons . Why? Because the share buttons is still part of the post (for sharing the post).

This is what I implement on my site.

screenshot-hanifmu com-2021 01 19-15_51_12

@adityatelange
Copy link
Owner Author

In my opinion, the prev-next post links would be nice if it is placed below of share buttons . Why? Because the share buttons is still part of the post (for sharing the post).

This is what I implement on my site.

screenshot-hanifmu com-2021 01 19-15_51_12

The design does match #183 :)

@muhammadhanif
Copy link
Contributor

In my opinion, the prev-next post links would be nice if it is placed below of share buttons . Why? Because the share buttons is still part of the post (for sharing the post).
This is what I implement on my site.
screenshot-hanifmu com-2021 01 19-15_51_12

The design does match #183 :)

absolutely yes 😁

by the way, Thank you for releasing this great theme 👍

@kdkasad
Copy link
Contributor

kdkasad commented Jan 19, 2021

In my opinion, the prev-next post links would be nice if it is placed below of share buttons . Why? Because the share buttons is still part of the post (for sharing the post).

This is what I implement on my site.

screenshot-hanifmu com-2021 01 19-15_51_12

Now that I've seen both ways, I agree with this layout.

@stale
Copy link

stale bot commented Jan 26, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the no-activity No activity since long time label Jan 26, 2021
@stale stale bot removed the no-activity No activity since long time label Jan 27, 2021
@sonarcloud
Copy link

sonarcloud bot commented Jan 27, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@adityatelange adityatelange merged commit 778ee39 into master Jan 27, 2021
@adityatelange adityatelange deleted the prev_next_post_links branch January 28, 2021 08:37
@adityatelange adityatelange added the enhancement New feature or improvement label Mar 22, 2021
kylethedeveloper pushed a commit to kylethedeveloper/hugo-PaperMod that referenced this pull request Feb 21, 2023
usage =>
in site config add 
Params:
   ShowPostNavLinks: true
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Next/Previous post or related posts under single layout
3 participants