https://codereview.appspot.com/258980045/diff/1/MoinMoin/themes/basic/templates/layout.html File MoinMoin/themes/basic/templates/layout.html (right): https://codereview.appspot.com/258980045/diff/1/MoinMoin/themes/basic/templates/layout.html#newcode260 MoinMoin/themes/basic/templates/layout.html:260: </script> I think script should be written in a ...
8 years, 9 months ago
(2015-08-04 16:57:51 UTC)
#1
8 years, 8 months ago
(2015-08-07 18:01:23 UTC)
#2
On 2015/08/04 16:57:51, sksaurabhkathpalia wrote:
>
https://codereview.appspot.com/258980045/diff/1/MoinMoin/themes/basic/templat...
> File MoinMoin/themes/basic/templates/layout.html (right):
>
>
https://codereview.appspot.com/258980045/diff/1/MoinMoin/themes/basic/templat...
> MoinMoin/themes/basic/templates/layout.html:260: </script>
> I think script should be written in a separate file. Ask thomas also regarding
> this
Set max-height property of the the div of quicklinks so that it does not float
out. You can set the max-height such that only 4 quicklinks are visible at one
time, so that user can scroll between the quicklinks. We also discussed the same
in the last meeting
Issue 258980045: Search widget for quicklinks in basic theme
Created 8 years, 9 months ago by yask123
Modified 8 years, 8 months ago
Reviewers: sksaurabhkathpalia
Base URL:
Comments: 2