add boostrab 3 submenu template #247
Conversation
| @@ -0,0 +1,14 @@ | |||
| {% load sitetree %} | |||
There was a problem hiding this comment.
Oh, that shouldn't be there, of course.
| @@ -0,0 +1,49 @@ | |||
| // Taken from https://codepen.io/ajaypatelaj/pen/prHjD | |||
There was a problem hiding this comment.
This probably should be minified. What do you think?
There was a problem hiding this comment.
I think, that we should keep the human-editable version of this file. So I can keep this and include also the minified version, although I don't think it is ideal due to code duplicity in the repo.
The ideal solution would be to compress the file in setup.py during packing.
Another possibility is to only suggest using django-compressor in the docs without dealing with the minification at all.
docs/source/templatesmod.rst
Outdated
| Constructs Bootstrap 3 menu with infinite submenus. | ||
| Requires adding extra CSS:: | ||
|
|
||
| <link href="/static/css/sitetree_bootstrap_submenu.css" type="text/css" rel="stylesheet" media="screen"> |
There was a problem hiding this comment.
/static/ shouldn't be harcoded, since it can change.
There was a problem hiding this comment.
Of course, that is mistake.
|
@idlesign I fixed the obvious errors and suggested three possibilities for the minification. Pleas choose the best option, and I will update the PR by that. |
|
Unminified will do. Merged. |
PR for #246