Nikola theme update and fixes by neofelis2X · Pull Request #11 · wxWidgets/wxPyWeb

neofelis2X · 2026-07-21T19:28:36Z

Hello,

I updated the nikola theme to bootstrap v5 so that the page renders well and looks nice.

This is how the updated version looks like: https://wxpyweb-test.netlify.app

Bootstrap 5.3.8 overwrites Nikolas internal bootstrap 4 theme
Fontawesome updated to 7.3.1
Cleaned up image of the logo
New favicon and apple-touch-icon
Reformatted the bookshelf page
Sidebar is now loaded from conf.py and not hardcoded in the theme
Fixed a bunch of broken links
Updated versions of the actions in the github workflow. This should get rid of the warning that Node.js 20 actions are deprecated.

Hopefully this is helpful and please let me know if there are any other changes you would like to have implemented.

echoix · 2026-07-23T07:08:32Z

It’s kinda hard to review a PR mixing multiple of different things, but at least you provide a live version to observe it working

echoix

I have a couple questions and assumptions to verify

echoix · 2026-07-23T07:12:25Z

+* Zip files with `*.pdb` files which are Visual Studio debugger information files.

-* Binary wheel files for a few flavors of **Linux**. 
+* Binary wheel files for a few flavours of **Linux**.


Before it was using US spelling, now it is UK/Canada/Australian spelling

okay, US spelling then

echoix · 2026-07-23T07:15:26Z

 # environment but is not recommended for HTTP/2.0 when caching is used.
 # Defaults to True.
-USE_BUNDLES = not USE_TEST_SITE
+# !This seems to brake boostrap v5


Brake or break?

echoix · 2026-07-23T07:16:14Z

 # It can be anything, data, functions, modules, etc.
 GLOBAL_CONTEXT = {
-    'header_logo': '<img src="/images/header-logo.png" />'
+    'example': '/images/header-logo.png'


I might understand later in the review, but is "example" expected here?

Nikola has a dedicated variable LOGO_URL that is now used.
I just left a dummy entry in GLOBAL_CONTEXT

echoix · 2026-07-23T07:19:30Z

+<!--
+.. title: Welcome to wxPython!
+.. slug: index
+.. tags:
+.. link:
+.. description: All about wxPython, the cross-platform GUI toolkit for the Python language
+-->


This is usually yaml, it is called "front-matter", where you add some metadata that the website builder uses when creating html pages from markdown. I often see it with a separator after the end of that section.

Nikola supports a variety of styles for front matter. This style is consistent with that of every other file in this folder.

echoix · 2026-07-23T07:22:27Z

 	 experiences, etc. All questions that are Python specific should be asked here.<br>
     [Mail Address](mailto:wxpython-users@googlegroups.com),
-     [Group URL](http://groups.google.com/group/wxpython-users)
+     [Group URL](https://groups.google.com/group/wxpython-users)


I assume here, and for all other http->https changes in this PR, that you tested that the server responds correctly. I still see some that don't work with https, which is another kind of problem.

Yes, i mainly checked for dead links but also changed them to https in case they redirected.

echoix · 2026-07-23T07:28:37Z

Usually, for screenshots (especially text), jpeg isn’t used because of the loss in quality of text and graphics. The gif used was an option that would be lossless if it fit in the palette restrictions, but there’s also .png that can be lossless and we often see in screenshots.

This applies to the multiple recompressed into jpeg files that are screenshots.

Websites usually care alot about filesizes. But I agree, in this case the lossless files should be kept.

echoix · 2026-07-23T07:32:06Z

+##
+## A shortcode template to add a book to the bookshelf
+##
+## Expected Parameters:
+##      asin: number of book
+##      title: title of book
+##      img: path to the bookcover
+##


What file type (syntax) does tmpl files use? (To make sure lines starting with # are parsed as comments)

Nikola uses the mako template engine, in this case.

single line comment, two ##:
## this is a comment.

multi line comment:

<%doc> these are comments more comments </%doc>

echoix · 2026-07-23T07:32:30Z

@@ -1,54 +1,159 @@
 /* 
- * wxPython specific overrides for style items in the base and bootstrap3 themes.
+ * wxPython specific overrides for style items in the base and bootstrap4 themes.


Bootstrap 4 or 5?

Nikola provides a base-theme that uses bootstrap 4. The custom wxPython theme (now) overwrites it with bootstrap 5.

echoix · 2026-07-23T07:34:23Z

 [Theme]
 engine = mako
-parent = bootstrap3
+parent = bootstrap4


Nikola provides a base-theme that uses bootstrap 4. The custom wxPython theme (now) overwrites it with bootstrap 5.

echoix · 2026-07-23T07:35:43Z


 # Name of the theme to use.
-#THEME = "bootstrap3"
+#THEME = "bootstrap4"


Nikola provides a base-theme that uses bootstrap 4. The custom wxPython theme (now) overwrites it with bootstrap 5.

neofelis2X · 2026-07-23T16:01:49Z

It’s kinda hard to review a PR mixing multiple of different things, but at least you provide a live version to observe it working

I see. I can split it into smaller PR, if you like, Just didn't want to spam you.

neofelis2X · 2026-07-26T14:30:58Z

I reverted the changed screenshots and the changed the two words back to US spelling. The incorrect comment in conf.py about USE_BUNDLES is now clarified.

neofelis2X added 6 commits July 21, 2026 20:06

Update theme to bootstrap 5, fontawesome 7

d22ab58

Update logo image and favicons

a0e4b0d

Reformat bookshelf page

5b82665

Update some hyperlinks

2f2b969

Update screenshot gallery

f22103f

Update github action and workflows

a049a12

echoix reviewed Jul 23, 2026

View reviewed changes

neofelis2X added 3 commits July 26, 2026 15:54

Revert changed screenshots to original gifs

33b77ec

fixup! Update some hyperlinks

450df5a

fixup! Update theme to bootstrap 5, fontawesome 7

848c3ce

Uh oh!

Conversation

neofelis2X commented Jul 21, 2026

Uh oh!

echoix commented Jul 23, 2026

Uh oh!

echoix left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

neofelis2X commented Jul 23, 2026

Uh oh!

neofelis2X commented Jul 26, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants