author | Lars Hjemli <hjemli@gmail.com> | 2008-04-13 10:20:00 (UTC) |
---|---|---|
committer | Lars Hjemli <hjemli@gmail.com> | 2008-04-13 10:20:00 (UTC) |
commit | 39912a24edf45497e668ebda25636aa6f6db0c9b (patch) (unidiff) | |
tree | 5de079ba1441a4f9546252e38c31d4d1b5f17b40 /ui-shared.c | |
parent | 536b0541fcfea2169e4df33043cd9ff14c657bce (diff) | |
download | cgit-39912a24edf45497e668ebda25636aa6f6db0c9b.zip cgit-39912a24edf45497e668ebda25636aa6f6db0c9b.tar.gz cgit-39912a24edf45497e668ebda25636aa6f6db0c9b.tar.bz2 |
Remove 'patch' link from tab, add to commit view
It's a bit confusing to enter the patch view from the tab, since it has no
layout. And the commit view has always lacked showing the commit id. Both of
these warts are fixed by this commit, which adds a new header line in the
commit view which shows the commit id as a 'permalink' to the current commit
and also adds a link to the patch view of the current commit.
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
-rw-r--r-- | ui-shared.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/ui-shared.c b/ui-shared.c index 6253a90..bb08c4a 100644 --- a/ui-shared.c +++ b/ui-shared.c | |||
@@ -352,260 +352,258 @@ void cgit_print_age(time_t t, time_t max_relative, char *format) | |||
352 | } | 352 | } |
353 | htmlf("<span class='age-years'>%.0f years</span>", | 353 | htmlf("<span class='age-years'>%.0f years</span>", |
354 | secs * 1.0 / TM_YEAR); | 354 | secs * 1.0 / TM_YEAR); |
355 | } | 355 | } |
356 | 356 | ||
357 | void cgit_print_http_headers(struct cgit_context *ctx) | 357 | void cgit_print_http_headers(struct cgit_context *ctx) |
358 | { | 358 | { |
359 | if (ctx->page.mimetype && ctx->page.charset) | 359 | if (ctx->page.mimetype && ctx->page.charset) |
360 | htmlf("Content-Type: %s; charset=%s\n", ctx->page.mimetype, | 360 | htmlf("Content-Type: %s; charset=%s\n", ctx->page.mimetype, |
361 | ctx->page.charset); | 361 | ctx->page.charset); |
362 | else if (ctx->page.mimetype) | 362 | else if (ctx->page.mimetype) |
363 | htmlf("Content-Type: %s\n", ctx->page.mimetype); | 363 | htmlf("Content-Type: %s\n", ctx->page.mimetype); |
364 | if (ctx->page.filename) | 364 | if (ctx->page.filename) |
365 | htmlf("Content-Disposition: inline; filename=\"%s\"\n", | 365 | htmlf("Content-Disposition: inline; filename=\"%s\"\n", |
366 | ctx->page.filename); | 366 | ctx->page.filename); |
367 | htmlf("Last-Modified: %s\n", http_date(ctx->page.modified)); | 367 | htmlf("Last-Modified: %s\n", http_date(ctx->page.modified)); |
368 | htmlf("Expires: %s\n", http_date(ctx->page.expires)); | 368 | htmlf("Expires: %s\n", http_date(ctx->page.expires)); |
369 | html("\n"); | 369 | html("\n"); |
370 | } | 370 | } |
371 | 371 | ||
372 | void cgit_print_docstart(struct cgit_context *ctx) | 372 | void cgit_print_docstart(struct cgit_context *ctx) |
373 | { | 373 | { |
374 | html(cgit_doctype); | 374 | html(cgit_doctype); |
375 | html("<html xmlns='http://www.w3.org/1999/xhtml' xml:lang='en' lang='en'>\n"); | 375 | html("<html xmlns='http://www.w3.org/1999/xhtml' xml:lang='en' lang='en'>\n"); |
376 | html("<head>\n"); | 376 | html("<head>\n"); |
377 | html("<title>"); | 377 | html("<title>"); |
378 | html_txt(ctx->page.title); | 378 | html_txt(ctx->page.title); |
379 | html("</title>\n"); | 379 | html("</title>\n"); |
380 | htmlf("<meta name='generator' content='cgit %s'/>\n", cgit_version); | 380 | htmlf("<meta name='generator' content='cgit %s'/>\n", cgit_version); |
381 | if (ctx->cfg.robots && *ctx->cfg.robots) | 381 | if (ctx->cfg.robots && *ctx->cfg.robots) |
382 | htmlf("<meta name='robots' content='%s'/>\n", ctx->cfg.robots); | 382 | htmlf("<meta name='robots' content='%s'/>\n", ctx->cfg.robots); |
383 | html("<link rel='stylesheet' type='text/css' href='"); | 383 | html("<link rel='stylesheet' type='text/css' href='"); |
384 | html_attr(ctx->cfg.css); | 384 | html_attr(ctx->cfg.css); |
385 | html("'/>\n"); | 385 | html("'/>\n"); |
386 | html("</head>\n"); | 386 | html("</head>\n"); |
387 | html("<body>\n"); | 387 | html("<body>\n"); |
388 | } | 388 | } |
389 | 389 | ||
390 | void cgit_print_docend() | 390 | void cgit_print_docend() |
391 | { | 391 | { |
392 | html("</div>\n</body>\n</html>\n"); | 392 | html("</div>\n</body>\n</html>\n"); |
393 | } | 393 | } |
394 | 394 | ||
395 | int print_branch_option(const char *refname, const unsigned char *sha1, | 395 | int print_branch_option(const char *refname, const unsigned char *sha1, |
396 | int flags, void *cb_data) | 396 | int flags, void *cb_data) |
397 | { | 397 | { |
398 | char *name = (char *)refname; | 398 | char *name = (char *)refname; |
399 | html_option(name, name, ctx.qry.head); | 399 | html_option(name, name, ctx.qry.head); |
400 | return 0; | 400 | return 0; |
401 | } | 401 | } |
402 | 402 | ||
403 | int print_archive_ref(const char *refname, const unsigned char *sha1, | 403 | int print_archive_ref(const char *refname, const unsigned char *sha1, |
404 | int flags, void *cb_data) | 404 | int flags, void *cb_data) |
405 | { | 405 | { |
406 | struct tag *tag; | 406 | struct tag *tag; |
407 | struct taginfo *info; | 407 | struct taginfo *info; |
408 | struct object *obj; | 408 | struct object *obj; |
409 | char buf[256], *url; | 409 | char buf[256], *url; |
410 | unsigned char fileid[20]; | 410 | unsigned char fileid[20]; |
411 | int *header = (int *)cb_data; | 411 | int *header = (int *)cb_data; |
412 | 412 | ||
413 | if (prefixcmp(refname, "refs/archives")) | 413 | if (prefixcmp(refname, "refs/archives")) |
414 | return 0; | 414 | return 0; |
415 | strncpy(buf, refname+14, sizeof(buf)); | 415 | strncpy(buf, refname+14, sizeof(buf)); |
416 | obj = parse_object(sha1); | 416 | obj = parse_object(sha1); |
417 | if (!obj) | 417 | if (!obj) |
418 | return 1; | 418 | return 1; |
419 | if (obj->type == OBJ_TAG) { | 419 | if (obj->type == OBJ_TAG) { |
420 | tag = lookup_tag(sha1); | 420 | tag = lookup_tag(sha1); |
421 | if (!tag || parse_tag(tag) || !(info = cgit_parse_tag(tag))) | 421 | if (!tag || parse_tag(tag) || !(info = cgit_parse_tag(tag))) |
422 | return 0; | 422 | return 0; |
423 | hashcpy(fileid, tag->tagged->sha1); | 423 | hashcpy(fileid, tag->tagged->sha1); |
424 | } else if (obj->type != OBJ_BLOB) { | 424 | } else if (obj->type != OBJ_BLOB) { |
425 | return 0; | 425 | return 0; |
426 | } else { | 426 | } else { |
427 | hashcpy(fileid, sha1); | 427 | hashcpy(fileid, sha1); |
428 | } | 428 | } |
429 | if (!*header) { | 429 | if (!*header) { |
430 | html("<h1>download</h1>\n"); | 430 | html("<h1>download</h1>\n"); |
431 | *header = 1; | 431 | *header = 1; |
432 | } | 432 | } |
433 | url = cgit_pageurl(ctx.qry.repo, "blob", | 433 | url = cgit_pageurl(ctx.qry.repo, "blob", |
434 | fmt("id=%s&path=%s", sha1_to_hex(fileid), | 434 | fmt("id=%s&path=%s", sha1_to_hex(fileid), |
435 | buf)); | 435 | buf)); |
436 | html_link_open(url, NULL, "menu"); | 436 | html_link_open(url, NULL, "menu"); |
437 | html_txt(strlpart(buf, 20)); | 437 | html_txt(strlpart(buf, 20)); |
438 | html_link_close(); | 438 | html_link_close(); |
439 | return 0; | 439 | return 0; |
440 | } | 440 | } |
441 | 441 | ||
442 | void add_hidden_formfields(int incl_head, int incl_search, char *page) | 442 | void add_hidden_formfields(int incl_head, int incl_search, char *page) |
443 | { | 443 | { |
444 | char *url; | 444 | char *url; |
445 | 445 | ||
446 | if (!ctx.cfg.virtual_root) { | 446 | if (!ctx.cfg.virtual_root) { |
447 | url = fmt("%s/%s", ctx.qry.repo, page); | 447 | url = fmt("%s/%s", ctx.qry.repo, page); |
448 | if (ctx.qry.path) | 448 | if (ctx.qry.path) |
449 | url = fmt("%s/%s", url, ctx.qry.path); | 449 | url = fmt("%s/%s", url, ctx.qry.path); |
450 | html_hidden("url", url); | 450 | html_hidden("url", url); |
451 | } | 451 | } |
452 | 452 | ||
453 | if (incl_head && strcmp(ctx.qry.head, ctx.repo->defbranch)) | 453 | if (incl_head && strcmp(ctx.qry.head, ctx.repo->defbranch)) |
454 | html_hidden("h", ctx.qry.head); | 454 | html_hidden("h", ctx.qry.head); |
455 | 455 | ||
456 | if (ctx.qry.sha1) | 456 | if (ctx.qry.sha1) |
457 | html_hidden("id", ctx.qry.sha1); | 457 | html_hidden("id", ctx.qry.sha1); |
458 | if (ctx.qry.sha2) | 458 | if (ctx.qry.sha2) |
459 | html_hidden("id2", ctx.qry.sha2); | 459 | html_hidden("id2", ctx.qry.sha2); |
460 | 460 | ||
461 | if (incl_search) { | 461 | if (incl_search) { |
462 | if (ctx.qry.grep) | 462 | if (ctx.qry.grep) |
463 | html_hidden("qt", ctx.qry.grep); | 463 | html_hidden("qt", ctx.qry.grep); |
464 | if (ctx.qry.search) | 464 | if (ctx.qry.search) |
465 | html_hidden("q", ctx.qry.search); | 465 | html_hidden("q", ctx.qry.search); |
466 | } | 466 | } |
467 | } | 467 | } |
468 | 468 | ||
469 | char *hc(struct cgit_cmd *cmd, const char *page) | 469 | char *hc(struct cgit_cmd *cmd, const char *page) |
470 | { | 470 | { |
471 | return (strcmp(cmd->name, page) ? NULL : "active"); | 471 | return (strcmp(cmd->name, page) ? NULL : "active"); |
472 | } | 472 | } |
473 | 473 | ||
474 | void cgit_print_pageheader(struct cgit_context *ctx) | 474 | void cgit_print_pageheader(struct cgit_context *ctx) |
475 | { | 475 | { |
476 | struct cgit_cmd *cmd = cgit_get_cmd(ctx); | 476 | struct cgit_cmd *cmd = cgit_get_cmd(ctx); |
477 | 477 | ||
478 | html("<table id='header'>\n"); | 478 | html("<table id='header'>\n"); |
479 | html("<tr>\n"); | 479 | html("<tr>\n"); |
480 | html("<td class='logo' rowspan='2'><a href='"); | 480 | html("<td class='logo' rowspan='2'><a href='"); |
481 | if (ctx->cfg.logo_link) | 481 | if (ctx->cfg.logo_link) |
482 | html_attr(ctx->cfg.logo_link); | 482 | html_attr(ctx->cfg.logo_link); |
483 | else | 483 | else |
484 | html_attr(cgit_rooturl()); | 484 | html_attr(cgit_rooturl()); |
485 | html("'><img src='"); | 485 | html("'><img src='"); |
486 | html_attr(ctx->cfg.logo); | 486 | html_attr(ctx->cfg.logo); |
487 | html("'/></a></td>\n"); | 487 | html("'/></a></td>\n"); |
488 | 488 | ||
489 | html("<td class='main'>"); | 489 | html("<td class='main'>"); |
490 | if (ctx->repo) { | 490 | if (ctx->repo) { |
491 | /* | 491 | /* |
492 | html("<a href='"); | 492 | html("<a href='"); |
493 | html_attr(cgit_rooturl()); | 493 | html_attr(cgit_rooturl()); |
494 | html("'>index</a> : "); | 494 | html("'>index</a> : "); |
495 | */ | 495 | */ |
496 | reporevlink(NULL, ctx->repo->name, NULL, hc(cmd, "summary"), | 496 | reporevlink(NULL, ctx->repo->name, NULL, hc(cmd, "summary"), |
497 | ctx->qry.head, NULL, NULL); | 497 | ctx->qry.head, NULL, NULL); |
498 | html(" : "); | 498 | html(" : "); |
499 | html_txt(ctx->qry.page); | 499 | html_txt(ctx->qry.page); |
500 | html("</td><td class='form'>"); | 500 | html("</td><td class='form'>"); |
501 | html("<form method='get' action=''>\n"); | 501 | html("<form method='get' action=''>\n"); |
502 | add_hidden_formfields(0, 1, ctx->qry.page); | 502 | add_hidden_formfields(0, 1, ctx->qry.page); |
503 | html("<select name='h' onchange='this.form.submit();'>\n"); | 503 | html("<select name='h' onchange='this.form.submit();'>\n"); |
504 | for_each_branch_ref(print_branch_option, ctx->qry.head); | 504 | for_each_branch_ref(print_branch_option, ctx->qry.head); |
505 | html("</select> "); | 505 | html("</select> "); |
506 | html("<input type='submit' name='' value='switch'/>"); | 506 | html("<input type='submit' name='' value='switch'/>"); |
507 | html("</form>"); | 507 | html("</form>"); |
508 | } else | 508 | } else |
509 | html_txt(ctx->cfg.root_title); | 509 | html_txt(ctx->cfg.root_title); |
510 | html("</td>\n"); | 510 | html("</td>\n"); |
511 | 511 | ||
512 | html("<tr><td class='sub'"); | 512 | html("<tr><td class='sub'"); |
513 | if (ctx->repo) { | 513 | if (ctx->repo) { |
514 | html(" colspan='2'>"); | 514 | html(" colspan='2'>"); |
515 | html_txt(ctx->repo->desc); | 515 | html_txt(ctx->repo->desc); |
516 | } | 516 | } |
517 | /* | 517 | /* |
518 | else if (ctx->cfg.root_subtitle) | 518 | else if (ctx->cfg.root_subtitle) |
519 | html_txt(ctx->cfg.root_subtitle); | 519 | html_txt(ctx->cfg.root_subtitle); |
520 | */ | 520 | */ |
521 | else { | 521 | else { |
522 | html(">"); | 522 | html(">"); |
523 | html_txt("a fast webinterface for the git dscm"); | 523 | html_txt("a fast webinterface for the git dscm"); |
524 | } | 524 | } |
525 | html("</td></tr>\n"); | 525 | html("</td></tr>\n"); |
526 | 526 | ||
527 | html("</tr>\n"); | 527 | html("</tr>\n"); |
528 | html("</table>\n"); | 528 | html("</table>\n"); |
529 | 529 | ||
530 | html("<table class='tabs'><tr><td>\n"); | 530 | html("<table class='tabs'><tr><td>\n"); |
531 | if (ctx->repo) { | 531 | if (ctx->repo) { |
532 | reporevlink(NULL, "summary", NULL, hc(cmd, "summary"), | 532 | reporevlink(NULL, "summary", NULL, hc(cmd, "summary"), |
533 | ctx->qry.head, NULL, NULL); | 533 | ctx->qry.head, NULL, NULL); |
534 | cgit_refs_link("refs", NULL, hc(cmd, "refs"), ctx->qry.head, | 534 | cgit_refs_link("refs", NULL, hc(cmd, "refs"), ctx->qry.head, |
535 | ctx->qry.sha1, NULL); | 535 | ctx->qry.sha1, NULL); |
536 | cgit_log_link("log", NULL, hc(cmd, "log"), ctx->qry.head, | 536 | cgit_log_link("log", NULL, hc(cmd, "log"), ctx->qry.head, |
537 | NULL, NULL, 0, NULL, NULL); | 537 | NULL, NULL, 0, NULL, NULL); |
538 | cgit_tree_link("tree", NULL, hc(cmd, "tree"), ctx->qry.head, | 538 | cgit_tree_link("tree", NULL, hc(cmd, "tree"), ctx->qry.head, |
539 | ctx->qry.sha1, NULL); | 539 | ctx->qry.sha1, NULL); |
540 | cgit_commit_link("commit", NULL, hc(cmd, "commit"), | 540 | cgit_commit_link("commit", NULL, hc(cmd, "commit"), |
541 | ctx->qry.head, ctx->qry.sha1); | 541 | ctx->qry.head, ctx->qry.sha1); |
542 | cgit_diff_link("diff", NULL, hc(cmd, "diff"), ctx->qry.head, | 542 | cgit_diff_link("diff", NULL, hc(cmd, "diff"), ctx->qry.head, |
543 | ctx->qry.sha1, ctx->qry.sha2, NULL); | 543 | ctx->qry.sha1, ctx->qry.sha2, NULL); |
544 | cgit_patch_link("patch", NULL, hc(cmd, "patch"), ctx->qry.head, | ||
545 | ctx->qry.sha1); | ||
546 | html("</td><td class='form'>"); | 544 | html("</td><td class='form'>"); |
547 | html("<form class='right' method='get' action='"); | 545 | html("<form class='right' method='get' action='"); |
548 | if (ctx->cfg.virtual_root) | 546 | if (ctx->cfg.virtual_root) |
549 | html_attr(cgit_fileurl(ctx->qry.repo, "log", | 547 | html_attr(cgit_fileurl(ctx->qry.repo, "log", |
550 | ctx->qry.path, NULL)); | 548 | ctx->qry.path, NULL)); |
551 | html("'>\n"); | 549 | html("'>\n"); |
552 | add_hidden_formfields(1, 0, "log"); | 550 | add_hidden_formfields(1, 0, "log"); |
553 | html("<select name='qt'>\n"); | 551 | html("<select name='qt'>\n"); |
554 | html_option("grep", "log msg", ctx->qry.grep); | 552 | html_option("grep", "log msg", ctx->qry.grep); |
555 | html_option("author", "author", ctx->qry.grep); | 553 | html_option("author", "author", ctx->qry.grep); |
556 | html_option("committer", "committer", ctx->qry.grep); | 554 | html_option("committer", "committer", ctx->qry.grep); |
557 | html("</select>\n"); | 555 | html("</select>\n"); |
558 | html("<input class='txt' type='text' size='10' name='q' value='"); | 556 | html("<input class='txt' type='text' size='10' name='q' value='"); |
559 | html_attr(ctx->qry.search); | 557 | html_attr(ctx->qry.search); |
560 | html("'/>\n"); | 558 | html("'/>\n"); |
561 | html("<input type='submit' value='search'/>\n"); | 559 | html("<input type='submit' value='search'/>\n"); |
562 | html("</form>\n"); | 560 | html("</form>\n"); |
563 | } else { | 561 | } else { |
564 | html("<a class='active' href='"); | 562 | html("<a class='active' href='"); |
565 | html_attr(cgit_rooturl()); | 563 | html_attr(cgit_rooturl()); |
566 | html("'>index</a>\n"); | 564 | html("'>index</a>\n"); |
567 | html("</td><td class='form'>"); | 565 | html("</td><td class='form'>"); |
568 | html("<form method='get' action='"); | 566 | html("<form method='get' action='"); |
569 | html_attr(cgit_rooturl()); | 567 | html_attr(cgit_rooturl()); |
570 | html("'>\n"); | 568 | html("'>\n"); |
571 | html("<input type='text' name='q' size='10' value='"); | 569 | html("<input type='text' name='q' size='10' value='"); |
572 | html_attr(ctx->qry.search); | 570 | html_attr(ctx->qry.search); |
573 | html("'/>\n"); | 571 | html("'/>\n"); |
574 | html("<input type='submit' value='search'/>\n"); | 572 | html("<input type='submit' value='search'/>\n"); |
575 | html("</form>"); | 573 | html("</form>"); |
576 | } | 574 | } |
577 | html("</td></tr></table>\n"); | 575 | html("</td></tr></table>\n"); |
578 | html("<div class='content'>"); | 576 | html("<div class='content'>"); |
579 | } | 577 | } |
580 | 578 | ||
581 | void cgit_print_filemode(unsigned short mode) | 579 | void cgit_print_filemode(unsigned short mode) |
582 | { | 580 | { |
583 | if (S_ISDIR(mode)) | 581 | if (S_ISDIR(mode)) |
584 | html("d"); | 582 | html("d"); |
585 | else if (S_ISLNK(mode)) | 583 | else if (S_ISLNK(mode)) |
586 | html("l"); | 584 | html("l"); |
587 | else if (S_ISGITLINK(mode)) | 585 | else if (S_ISGITLINK(mode)) |
588 | html("m"); | 586 | html("m"); |
589 | else | 587 | else |
590 | html("-"); | 588 | html("-"); |
591 | html_fileperm(mode >> 6); | 589 | html_fileperm(mode >> 6); |
592 | html_fileperm(mode >> 3); | 590 | html_fileperm(mode >> 3); |
593 | html_fileperm(mode); | 591 | html_fileperm(mode); |
594 | } | 592 | } |
595 | 593 | ||
596 | void cgit_print_snapshot_links(const char *repo, const char *head, | 594 | void cgit_print_snapshot_links(const char *repo, const char *head, |
597 | const char *hex, int snapshots) | 595 | const char *hex, int snapshots) |
598 | { | 596 | { |
599 | const struct cgit_snapshot_format* f; | 597 | const struct cgit_snapshot_format* f; |
600 | char *filename; | 598 | char *filename; |
601 | 599 | ||
602 | for (f = cgit_snapshot_formats; f->suffix; f++) { | 600 | for (f = cgit_snapshot_formats; f->suffix; f++) { |
603 | if (!(snapshots & f->bit)) | 601 | if (!(snapshots & f->bit)) |
604 | continue; | 602 | continue; |
605 | filename = fmt("%s-%s%s", cgit_repobasename(repo), hex, | 603 | filename = fmt("%s-%s%s", cgit_repobasename(repo), hex, |
606 | f->suffix); | 604 | f->suffix); |
607 | cgit_snapshot_link(filename, NULL, NULL, (char *)head, | 605 | cgit_snapshot_link(filename, NULL, NULL, (char *)head, |
608 | (char *)hex, filename); | 606 | (char *)hex, filename); |
609 | html("<br/>"); | 607 | html("<br/>"); |
610 | } | 608 | } |
611 | } | 609 | } |