diff options
author | Lukas Fleischer | 2013-03-07 08:56:22 +0100 |
---|---|---|
committer | Jason A. Donenfeld | 2013-03-20 21:16:19 +0100 |
commit | b60e6bff75719a5fb0df970bac3be6b2726cf73a (patch) | |
tree | 706c27228919f3cca2d307b1d3dfcc561b0cd64e /ui-log.c | |
parent | 121089ced5e1d3f3103cbc2b37f5fb579d800915 (diff) | |
download | cgit-b60e6bff75719a5fb0df970bac3be6b2726cf73a.tar.gz cgit-b60e6bff75719a5fb0df970bac3be6b2726cf73a.tar.bz2 cgit-b60e6bff75719a5fb0df970bac3be6b2726cf73a.zip |
Convert pager navigation into a unordered list
It is common practice and semantically appropriate to use unordered
lists for long navigation lists.
This also fixes the layout of very long pager navigations in
Webkit-based browsers.
Signed-off-by: Lukas Fleischer <cgit@cryptocrack.de>
Diffstat (limited to 'ui-log.c')
-rw-r--r-- | ui-log.c | 9 |
1 files changed, 6 insertions, 3 deletions
@@ -405,21 +405,24 @@ void cgit_print_log(const char *tip, int ofs, int cnt, char *grep, char *pattern | |||
405 | commit->parents = NULL; | 405 | commit->parents = NULL; |
406 | } | 406 | } |
407 | if (pager) { | 407 | if (pager) { |
408 | html("</table><div class='pager'>"); | 408 | html("</table><ul class='pager'>"); |
409 | if (ofs > 0) { | 409 | if (ofs > 0) { |
410 | html("<li>"); | ||
410 | cgit_log_link("[prev]", NULL, NULL, ctx.qry.head, | 411 | cgit_log_link("[prev]", NULL, NULL, ctx.qry.head, |
411 | ctx.qry.sha1, ctx.qry.vpath, | 412 | ctx.qry.sha1, ctx.qry.vpath, |
412 | ofs - cnt, ctx.qry.grep, | 413 | ofs - cnt, ctx.qry.grep, |
413 | ctx.qry.search, ctx.qry.showmsg); | 414 | ctx.qry.search, ctx.qry.showmsg); |
414 | html(" "); | 415 | html("</li>"); |
415 | } | 416 | } |
416 | if ((commit = get_revision(&rev)) != NULL) { | 417 | if ((commit = get_revision(&rev)) != NULL) { |
418 | html("<li>"); | ||
417 | cgit_log_link("[next]", NULL, NULL, ctx.qry.head, | 419 | cgit_log_link("[next]", NULL, NULL, ctx.qry.head, |
418 | ctx.qry.sha1, ctx.qry.vpath, | 420 | ctx.qry.sha1, ctx.qry.vpath, |
419 | ofs + cnt, ctx.qry.grep, | 421 | ofs + cnt, ctx.qry.grep, |
420 | ctx.qry.search, ctx.qry.showmsg); | 422 | ctx.qry.search, ctx.qry.showmsg); |
423 | html("</li>"); | ||
421 | } | 424 | } |
422 | html("</div>"); | 425 | html("</ul>"); |
423 | } else if ((commit = get_revision(&rev)) != NULL) { | 426 | } else if ((commit = get_revision(&rev)) != NULL) { |
424 | htmlf("<tr class='nohover'><td colspan='%d'>", columns); | 427 | htmlf("<tr class='nohover'><td colspan='%d'>", columns); |
425 | cgit_log_link("[...]", NULL, NULL, ctx.qry.head, NULL, | 428 | cgit_log_link("[...]", NULL, NULL, ctx.qry.head, NULL, |