bunch of updates
authorIan Kelling <iank@fsf.org>
Mon, 5 Aug 2019 20:58:06 +0000 (16:58 -0400)
committerIan Kelling <iank@fsf.org>
Tue, 6 Aug 2019 03:35:37 +0000 (23:35 -0400)
For blogs, move to left justified, make code sections expand over the
whole screen so they aren't cut off. And on mobile, make them scrollable
because otherwise they overflow the body and i dunno how to fix that.

Fix email error 500 error on comment submission, hidden form not being
hidden. Change main content background to be darker so code sections
stand out more so you notice you can scroll them.

Change comment box to be more standard white with no text in it so its
easier to use.

.gitignore
README
_site/cgi/comment
b.rb
blog/2014-09-29-say-on2.md
css/common.scss
css/main.scss
set-comments

index f7e2e83..ed9a601 100644 (file)
@@ -11,3 +11,4 @@
 
 /resume.md
 /_site/resume.html
+/_site/tmp
diff --git a/README b/README
index 03e77b4..7957f6e 100644 (file)
--- a/README
+++ b/README
@@ -6,7 +6,6 @@ Requires a modern debian based OS, tested on debian 8 & 9.
 
 Some programs need to be in PATH:
 https://iankelling.org/git/?p=basic-https-conf;a=summary
-https://iankelling.org/git/?p=acme-tiny-wrapper;a=summary
 chost from
 https://iankelling.org/git/?p=fai;a=summary
 
index b84c45e..1497994 100755 (executable)
@@ -26,7 +26,6 @@ require 'cgi'
 require 'fileutils'
 require 'time'
 require 'sqlite3'
-
 Dir.chdir(File.join(File.dirname(__FILE__), '..'))
 
 require '../b'
@@ -85,7 +84,7 @@ def do_captcha
 <p>#{captcha_q}</p>
 
 <form action="/cgi/comment" method="post">
-  <input class="misc" type="text" name="url">
+  <input class="misc-comment-input" type="text" name="url">
   <input name="goto" type="hidden" value="#{GOTO}">
   <input name="question" type="hidden" value="#{captcha_q}">
   <input name="answer">
@@ -312,27 +311,19 @@ new_count = new_count[0][0]
 
 if new_count == 1
   require 'net/smtp'
-  def send_email(opts={})
-    opts[:to]          ||= 'root'
-    opts[:server]      ||= 'localhost'
-    opts[:from]        ||= 'root'
-    opts[:from_alias]  ||= 'root'
-    opts[:subject]     ||= "test subject"
-    opts[:body]        ||= ""
-
-    msg = <<END_OF_MESSAGE
-From: #{opts[:from_alias]} <#{opts[:from]}>
-To: <#{opts[:to]}>
-Subject: #{opts[:subject]}
-
-#{opts[:body]}
+  to = 'root@' + FQDN
+  from = 'www-data@' + FQDN
+  server = 'localhost'
+  msg = <<END_OF_MESSAGE
+From: h <#{from}>
+To: <#{to}>
+Subject: "new comment on #{FQDN}"
+
+empty body
 END_OF_MESSAGE
-
-    Net::SMTP.start(opts[:server]) do |smtp|
-      smtp.send_message msg, opts[:from], opts[:to]
-    end
+  Net::SMTP.start(server) do |smtp|
+    smtp.send_message msg, from, to
   end
-  send_email :subject => "new comment on #{DN}"
 end
 
 post(md_file)
diff --git a/b.rb b/b.rb
index 391a0ed..4bd8f96 100644 (file)
--- a/b.rb
+++ b/b.rb
@@ -86,7 +86,7 @@ EOF
       <h3><a href="/">iankelling.org</a>#{o[:header]}</h3>
     </header>
     <div class="main-content-stripe">
-      <div class="content">
+      <div class="#{o[:prose] ? "prose" : "content"}">
         #{content}
       </div>
     </div>
@@ -223,8 +223,8 @@ EOF
       <form action="/cgi/comment" method="post">
         <input class="misc-comment-input" type="text" name="url">
         <input name="goto" type="hidden" value="#{rel_path}">
-        <textarea rows="10" name="comment" placeholder="markdown" maxlength="1000"></textarea>
-        <input type="submit" value="Leave a comment">
+        <textarea rows="10" name="comment" maxlength="1000"></textarea>
+        <input type="submit" value="Add a comment (markdown works)">
       </form>
       <div id="comments">
          #{com_list}
@@ -267,6 +267,11 @@ EOF
     fskel(rel_path, title, page_html,
           header: header_rel,
           footer: footer_extra,
+          # We call the main content prose because it can contain
+          # code, so we want to left justify it, so we dont have
+          # to cram all the code into the middle and can use
+          # a wider area.
+          prose: true,
           comments: com_section,
           description: description)
     url="#{DURL}#{rel_path}"
index 10b2119..2853a79 100644 (file)
@@ -163,8 +163,7 @@ def bnf_expand(*args):
 asymtotic = ['asymtotic', '']
 asymtotic_modified = ['bound', 'run time', 'growth', 'growth rate']
 commonly_modified = ['complexity', 'efficiency', 'growth rate']
-common_modifiers = ['computational', 'big oh', 'algorithmic', 'time',
-                     'asymtotic', '']
+common_modifiers = ['computational', 'big oh', 'algorithmic', 'time', 'asymtotic', '']
 unmodified = ['big oh', 'order', 'order of complexity', 'time']
 O = bnf_expand([asymtotic, asymtotic_modified],
                 [common_modifiers, commonly_modified], [unmodified])
index 8854f6d..f9370b2 100644 (file)
@@ -1,9 +1,9 @@
 // a few minor things are in main.css and gitweb.css
 
-$comment-background-color: #ebeae4;
+$comment-background-color: #d9d8d1;
 $comment-br-color: #c4c3bc; // a darkish color, black is too striking
 $comment-date-color: #555; // a bit lighter for less emphasis
-$content-color: #f6f6f0; // from gitweb.
+$content-color: #ebeae4;
 $inline-code-background-color: #fff;
 $link-color: #0645ad;
 $link-visited-color: #0b0080;
index e79d352..5e8876a 100644 (file)
   padding-bottom: 30px;
   padding-top: 30px;
 
-  textarea {
-    background-color: $content-color; // go with theme, avoid bright white.
-  }
 
   input:hover,
-  textarea:active,
-  textarea:focus {
+  textarea {
     // color textarea when selected, and for submit button,
     // without 3d effect, we need a different color.
     background: $text-input-color;
@@ -71,7 +67,7 @@ ul {
 li {
   // to compensate for no bullet points, we need extra space between list items
   // in the case of small screens where they wrap.
-  margin-top: 12px;
+  margin-top: 10px;
 }
 
 .site-nav-list {
@@ -137,18 +133,35 @@ footer {
 
 .comment-section,
 .content,
+.prose,
 .page_header,
 footer {
   // auto does the centering magic
   margin-bottom: 0;
-  margin-left: auto;
   margin-right: auto;
   margin-top: 0;
+  padding-left: 16px;
+  padding-right: 16px;
+}
+
+.content,
+.prose,
+.page_header,
+footer {
+  // auto on left +right does the centering magic
+  margin-left: auto;
+}
+
+
+.comment-section,
+.content,
+.prose :not(.highlight),
+.page_header,
+footer {
   max-width: $main-column-max-width; // default too wide
-  padding-left: 30px;
-  padding-right: 30px;
 }
 
+
 code {
   background-color: $inline-code-background-color;
   // defaults are squished next to the borders of the color change
@@ -161,6 +174,11 @@ pre {
   font-size: 15px;
 }
 
+// for code, scroll right instead of overflowing the body
+.highlight {
+  overflow: auto;
+}
+
 pre {
   // defaults are squished next to the borders of the color change
   padding: 6px 8px;
@@ -180,4 +198,3 @@ a:hover {
 a:visited {
   color: $link-visited-color;
 }
-
index b87f211..c503aec 100755 (executable)
@@ -23,7 +23,6 @@ cd "${BASH_SOURCE%/*}/proposed-comments"
 shell="ssh -oStrictHostKeyChecking=no $(chost iankelling.org)"
 while [[ $1 ]]; do
     case $1 in
-        -a) all=true; shift ;;
         -l) shell= ; shift ;; # -l  for local
     esac
 done