* [PATCH 4/4] viewvcs: wire up syntax-highlighting for blobs
2019-01-27 11:48 7% [PATCH 0/4] viewvcs: wire up highlight to solver Eric Wong
@ 2019-01-27 11:48 6% ` Eric Wong
0 siblings, 0 replies; 2+ results
From: Eric Wong @ 2019-01-27 11:48 UTC (permalink / raw)
To: meta
And update 216dark.css to match a color scheme I'm used to;
which is fairly minimal and doesn't use all the classes
"highlight" provides.
---
contrib/css/216dark.css | 20 ++++++++++++++++++++
lib/PublicInbox/UserContent.pm | 20 ++++++++++++++++++++
lib/PublicInbox/ViewVCS.pm | 15 ++++++++++++++-
3 files changed, 54 insertions(+), 1 deletion(-)
diff --git a/contrib/css/216dark.css b/contrib/css/216dark.css
index 2fd85d0..35ef7aa 100644
--- a/contrib/css/216dark.css
+++ b/contrib/css/216dark.css
@@ -24,3 +24,23 @@ a:visited { color:#96f }
*.del { color:#f0f }
*.head { color:#fff }
*.hunk { color:#c93 }
+
+/*
+ * highlight 3.x colors (tested 3.18)
+ * this doesn't use most of the colors available (I find too many
+ * colors overwhelming). So the #ccc default is commented out.
+ */
+.hl.num { color:#f30 }
+.hl.esc { color:#f0f }
+.hl.str { color:#f30 }
+.hl.pps { color:#f30 }
+/* .hl.slc { color:#ccc } */
+.hl.com { color:#09f }
+.hl.ppc { color:#f0f }
+/* .hl.opt { color:#ccc } */
+/* .hl.ipl { color:#ccc } */
+/* .hl.lin { color:#ccc } */
+.hl.kwa { color:#ff0 }
+.hl.kwb { color:#0ff }
+.hl.kwc { color:#ff0 }
+/* .hl.kwd { color:#ccc } */
diff --git a/lib/PublicInbox/UserContent.pm b/lib/PublicInbox/UserContent.pm
index b34ebf9..514cd95 100644
--- a/lib/PublicInbox/UserContent.pm
+++ b/lib/PublicInbox/UserContent.pm
@@ -36,6 +36,26 @@ sub CSS () {
*.del { color:#f0f }
*.head { color:#fff }
*.hunk { color:#c93 }
+
+ /*
+ * highlight 3.x colors (tested 3.18)
+ * this doesn't use most of the colors available (I find too many
+ * colors overwhelming). So the #ccc default is commented out.
+ */
+ .hl.num { color:#f30 }
+ .hl.esc { color:#f0f }
+ .hl.str { color:#f30 }
+ .hl.pps { color:#f30 }
+ /* .hl.slc { color:#ccc } */
+ .hl.com { color:#09f }
+ .hl.ppc { color:#f0f }
+ /* .hl.opt { color:#ccc } */
+ /* .hl.ipl { color:#ccc } */
+ /* .hl.lin { color:#ccc } */
+ .hl.kwa { color:#ff0 }
+ .hl.kwb { color:#0ff }
+ .hl.kwc { color:#ff0 }
+ /* .hl.kwd { color:#ccc } */
_
}
# end of auto-updated sub
diff --git a/lib/PublicInbox/ViewVCS.pm b/lib/PublicInbox/ViewVCS.pm
index 5de37ee..a8aa0b6 100644
--- a/lib/PublicInbox/ViewVCS.pm
+++ b/lib/PublicInbox/ViewVCS.pm
@@ -21,6 +21,11 @@ use PublicInbox::SolverGit;
use PublicInbox::WwwStream;
use PublicInbox::Linkify;
use PublicInbox::Hval qw(ascii_html to_filename);
+my $hl = eval {
+ require PublicInbox::HlMod;
+ PublicInbox::HlMod->new;
+};
+
my %QP_MAP = ( A => 'oid_a', B => 'oid_b', a => 'path_a', b => 'path_b' );
my $max_size = 1024 * 1024; # TODO: configurable
my $enc_utf8 = find_encoding('UTF-8');
@@ -88,6 +93,14 @@ sub solve_result {
my $nl = ($$blob =~ tr/\n/\n/);
my $pad = length($nl);
+ $l->linkify_1($$blob);
+ my $ok = $hl->do_hl($blob, $path) if $hl;
+ if ($ok) {
+ $blob = $ok;
+ } else {
+ $$blob = ascii_html($$blob);
+ }
+
# using some of the same CSS class names and ids as cgit
$log = "<pre>$oid $type $size bytes $raw_link</pre>" .
"<hr /><table\nclass=blob>".
@@ -96,7 +109,7 @@ sub solve_result {
} (1..$nl)) . '</pre></td>' .
'<td><pre> </pre></td>'. # pad for non-CSS users
"<td\nclass=lines><pre\nstyle='white-space:pre'><code>" .
- ascii_html($$blob) .
+ $l->linkify_2($$blob) .
'</code></pre></td></tr></table>' . $log;
html_page($ctx, 200, \$log);
--
EW
^ permalink raw reply related [relevance 6%]
* [PATCH 0/4] viewvcs: wire up highlight to solver
@ 2019-01-27 11:48 7% Eric Wong
2019-01-27 11:48 6% ` [PATCH 4/4] viewvcs: wire up syntax-highlighting for blobs Eric Wong
0 siblings, 1 reply; 2+ results
From: Eric Wong @ 2019-01-27 11:48 UTC (permalink / raw)
To: meta
Since we go through the trouble of recreating blobs with solver;
show them with syntax-highlighting.
Fortunately, Debian systems have "libhighlight-perl" SWIG bindings
packaged. Going to hammer-away on it with check-www-inbox.perl
before deploying
Eric Wong (4):
wwwhighlight: read_in_full returns undef on errors
t/hl_mod: extra check to ensure we escape HTML
hlmod: disable enclosing <pre> tag
viewvcs: wire up syntax-highlighting for blobs
contrib/css/216dark.css | 20 ++++++++++++++++++++
lib/PublicInbox/HlMod.pm | 1 -
lib/PublicInbox/UserContent.pm | 20 ++++++++++++++++++++
lib/PublicInbox/ViewVCS.pm | 15 ++++++++++++++-
lib/PublicInbox/WwwHighlight.pm | 1 +
t/hl_mod.t | 3 ++-
6 files changed, 57 insertions(+), 3 deletions(-)
--
EW
^ permalink raw reply [relevance 7%]
Results 1-2 of 2 | reverse | options above
-- pct% links below jump to the message on this page, permalinks otherwise --
2019-01-27 11:48 7% [PATCH 0/4] viewvcs: wire up highlight to solver Eric Wong
2019-01-27 11:48 6% ` [PATCH 4/4] viewvcs: wire up syntax-highlighting for blobs Eric Wong
Code repositories for project(s) associated with this public inbox
https://80x24.org/public-inbox.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).