Fossil

Check-in [ef5424b6]
Login

Many hyperlinks are disabled.
Use anonymous login to enable hyperlinks.

Overview
Comment:Added a parameter to style_footer() and renamed it to match its actual function (style_body_and_footer()) so we can add a CSS class to each page that indicates which major Fossil feature served the page, so we can have module-specific CSS.
Downloads: Tarball | ZIP archive
Timelines: family | ancestors | descendants | both | default-css-cleanups
Files: files | file ages | folders
SHA3-256: ef5424b683ad2830bfb111ea5756e00e3b8be5df5692ec95d28f1d851170c7c5
User & Date: wyoung 2020-11-05 23:15:41.862
About

There are several non-obvious things about this commit.

The rules I used for picking the "page type" for each call to style_body_and_footer() are:

  1. Collect multi-page features under a single page type: e.g. all "forum" pages use the same type.

  2. All "test" type pages are type "test", no matter where they're served from, so you can do something type-specific with them, like give the page title a different color. If this is later considered a design error, one could change these instances to, e.g. "backlink test" for the case in backlink.c.

  3. Where there are multiple pages that don't compose a major user-facing feature, I sometimes used the name of the file where those pages are collected as the common type for pages served from that file. e.g. "cache"

Other points of interest:

  1. The implementation of proc styleFooter in TH1 should probably have an optional parameter added to pass along to style_body_and_footer() rather than hard-code "th1".

  2. Since none of these names are currently used for anything, they can all change until someone locks them down by making use of them. I plan only to use "forum" in a subsequent commit. Everything else is up in the air.

Context
2020-11-05
23:41
Using the new "forum" page type to simplify repetitive CSS rules. ... (check-in: 064c677a user: wyoung tags: default-css-cleanups)
23:15
Added a parameter to style_footer() and renamed it to match its actual function (style_body_and_footer()) so we can add a CSS class to each page that indicates which major Fossil feature served the page, so we can have module-specific CSS. ... (check-in: ef5424b6 user: wyoung tags: default-css-cleanups)
22:34
Moved a comment down in src/style.c and removed a part of it that seems to refer to inline CSS that's since moved to style.css. ... (check-in: a040a5d2 user: wyoung tags: default-css-cleanups)
Changes
Unified Diff Side-by-Side Diff Patch
Changes to src/alerts.c.
Changes to src/attach.c.
Changes to src/backlink.c.
Changes to src/branch.c.
Changes to src/browse.c.
Changes to src/builtin.c.
Changes to src/cache.c.
Changes to src/captcha.c.
Changes to src/clone.c.
Changes to src/cookies.c.
Changes to src/descendants.c.
Changes to src/diff.c.
Changes to src/dispatch.c.
Changes to src/doc.c.
Changes to src/event.c.
Changes to src/extcgi.c.
Changes to src/fileedit.c.
Changes to src/finfo.c.
Changes to src/forum.c.
Changes to src/info.c.
Changes to src/interwiki.c.
Changes to src/loadctrl.c.
Changes to src/login.c.
Changes to src/main.c.
Changes to src/moderate.c.
Changes to src/name.c.
Changes to src/path.c.
Changes to src/piechart.c.
Changes to src/pikchrshow.c.
Changes to src/printf.c.
Changes to src/repolist.c.
Changes to src/report.c.
Changes to src/search.c.
Changes to src/security_audit.c.
Changes to src/setup.c.
Changes to src/setupuser.c.
Changes to src/shun.c.
Changes to src/sitemap.c.
Changes to src/skins.c.
Changes to src/smtp.c.
Changes to src/stat.c.
Changes to src/statrep.c.
Changes to src/style.c.
Changes to src/tag.c.
Changes to src/tar.c.
Changes to src/th_main.c.
Changes to src/timeline.c.
Changes to src/tkt.c.
Changes to src/tktsetup.c.
Changes to src/unversioned.c.
Changes to src/user.c.
Changes to src/webmail.c.
Changes to src/wiki.c.
Changes to src/xfersetup.c.
Changes to src/zip.c.