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

Add the ability to link static elements in the nav #28

Merged
merged 1 commit into from
May 14, 2018

Conversation

mateimicu
Copy link
Contributor

Basically you can define something like this

[[params.static]]                                                                                                                                                                                            
title = "Slides"
url = "http://slides.mateimicu.com"

and it will show up like this:
image

And of course you can add multiple links:

[[params.static]]                                                                                                                                                                                            
title = "Slides"
url = "http://slides.mateimicu.com"
[[params.static]]                                                                                                                                                                                            
title = "Slides2"
url = "http://slides.mateimicu.com/2"

image

@ribice
Copy link
Owner

ribice commented Apr 27, 2018

Can you document these changes in Readme? Thanks!

@mateimicu mateimicu force-pushed the master branch 2 times, most recently from 4f8a2d4 to 5208078 Compare April 27, 2018 20:36
@mateimicu
Copy link
Contributor Author

Added a little section with a config sample.
Would you like me to add a picture ?

@ribice
Copy link
Owner

ribice commented Apr 27, 2018

Not needed. I'll probably edit the text a bit tomorrow, and add it to example site as well.

@russparsloe
Copy link
Contributor

This seems similar to sections (#23) which uses this configuration format:

[params.sections]
  "/about" = "About"
  "/posts" = "Posts"

Can this be altered to make both options consistent in how they are configured in config.toml?

@ribice
Copy link
Owner

ribice commented Apr 28, 2018

@russparsloe That would be great!

@mateimicu
Copy link
Contributor Author

I'll make the change today :D

@ribice
Copy link
Owner

ribice commented May 4, 2018

@mateimicu: Any update?

@mateimicu
Copy link
Contributor Author

Updated the PR

@ribice ribice merged commit c934577 into ribice:master May 14, 2018
sttmt pushed a commit to sttmt/kiss that referenced this pull request May 16, 2018
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 this pull request may close these issues.

None yet

3 participants