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

Ask the designer how we should implement navigation between blog posts #28

Closed
josecelano opened this issue Sep 24, 2024 · 1 comment · Fixed by #44
Closed

Ask the designer how we should implement navigation between blog posts #28

josecelano opened this issue Sep 24, 2024 · 1 comment · Fixed by #44
Assignees

Comments

@josecelano
Copy link
Member

Current:

image

Expected:

It was not included in the original design.

@josecelano
Copy link
Member Author

Fer has added the design to Figma:

image

cc @grmbyrn

@josecelano josecelano linked a pull request Sep 25, 2024 that will close this issue
josecelano added a commit that referenced this issue Sep 25, 2024
19437dd Fix conflict issues (Graeme Byrne)
5c2e1df fix issues with v2 (Graeme Byrne)

Pull request description:

  * moved posts for index and tracker into `TorrustIndexPost.svelte` and `TorrustTrackerPost.svelte`
  * [Key features list width should be 100% #6](#6)
  * [Review design for table of contents #8](#8)
  * [Minor bugs in homepage #12](#12)
  * [Homepage cards don't have the same padding as design #25](#25)
  * [Top menu padding is not the same as the design #26](#26)
  * [Related Post section does not look like the standard blog post list #27](#27)
  * [Ask the designer how we should implement navigation between blog posts #28](#28)
  * [Ask the designer how to implement share links in blog posts #29](#29)
  * [Remove # prefix from titles #30](#30)
  * [Ask the designer how to implement highlighted terms or sentences in markdown #36](#36)
  * [Font style in feature card is not the same as in the original design #43](#43)

ACKs for top commit:
  josecelano:
    ACK 19437dd

Tree-SHA512: a60b8ee60753f8517ddd53ae1073d97b481b9badbc75938f364b0c239ab561a6b2eece7d01f5c4ff4e97af9804facfb04e8266377a32ce4a27c531dcf5263044
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants