TODO.md

@@ -1,6 +1,6 @@
 # To do
 
-- [ ] Move to a later C standard (C11?) and use safer string handling functions provided by it.
+- [ ] Settle on a C standard (C11?), and use safer string handling functions provided by it.
   - The thing is, I kinda feel attached to C89-C99
 - [ ] Consider
   - See make lint for purported insecurities
@@ -18,7 +18,6 @@
 
 # Previously done
 
-- [x] Check that this compiles with the c99 standard
 - [x] Add minimalist version of rosenrot to its own branch
 - [x] Fix PageUp/PageDown shortcuts.
 - ~~[ ] Set [`webkit_web_context_set_sandbox_enabled`](<https://webkitgtk.org/reference/webkit2gtk/2.36.8/WebKitWebContext.html#webkit-web-context-set-sandbox-enabled>), as recommended [here](<https://blogs.gnome.org/mcatanzaro/2022/11/04/stop-using-qtwebkit/>)~~. Irrelevant with upgrade to libsoup3.

config.h

@@ -86,10 +86,8 @@ static struct {
     { CTRL,        KEY(equal),         zoomin             },
     { CTRL,        KEY(minus),         zoomout            },
     { CTRL,        KEY(0),             zoom_reset         },
-    { CTRL,        KEY(KP_Page_Up),    prev_tab           }, 
-    { CTRL,        KEY(KP_Page_Down),  next_tab           }, 
-    { CTRL,        KEY(Page_Up),       prev_tab           }, // working hypothesis: Page_UP vs KP_Page_Up might depend on whether the user has a numpad
-    { CTRL,        KEY(Page_Down),     next_tab           }, 
+    { CTRL,        KEY(KP_Page_Up),    prev_tab           }, /* also try KEY(Page_Up) if this doesn't work on your machine */
+    { CTRL,        KEY(KP_Page_Down),  next_tab           }, /* ditto for KEY(Page_Down) */
     { CTRL,        KEY(t),             new_tab            },
     { CTRL,        KEY(w),             close_tab          },
     { 0x0,         KEY(F11),           toggle_fullscreen  },

plugins/style/style.js

@@ -160,15 +160,16 @@ switch (document.domain) {
         display: none !important;
       }
 
-      /* 
-      No change of colors in hover: seemed like a good idea
-      but it fucks up going back and forth
+      /* No change of colors in hover */
       *:hover {
-        background-color: white !important; 
-        background-color: inherit !important; 
+        /* background-color: white !important; */
         transition: none !important;
-      }
-      */
+      }*/
+      /*
+      *:hover {
+        background-color: inherit !important;
+        transition: none !important;
+      }*/
       /* Hide go to top button */
       [aria-label^="New posts are available. Push the period key to go to the them."]{
 		    display: none;

rosenrot.c

@@ -37,8 +37,8 @@ void load_uri(WebKitWebView* view, const char* uri)
     } else if (g_str_has_prefix(uri, "http://") || g_str_has_prefix(uri, "https://") || g_str_has_prefix(uri, "file://") || g_str_has_prefix(uri, "about:")) {
         webkit_web_view_load_uri(view, uri);
     } else if (strstr(uri, ".com") || strstr(uri, ".org")) {
-        char tmp[strlen("https://") + strlen(uri) + 1];
-        snprintf(tmp, sizeof(tmp) + 1, "https://%s", uri);
+        char tmp[strlen("https://") + strlen(uri)];
+        snprintf(tmp, sizeof(tmp), "https://%s", uri);
         webkit_web_view_load_uri(view, tmp);
     } else {
         // Check for shortcuts