fix/events-wiki #5628

Merged
tobiasd merged 4 commits from fix/events-wiki into master 2026-01-27 10:32:28 +00:00
Member

fixes #5622

fixes https://git.fsfe.org/FSFE/fsfe-website/issues/5622
delliott requested review from mk 2026-01-15 15:09:32 +00:00
delliott force-pushed fix/events-wiki from 5a8b949443 to 19f3e65716 2026-01-15 15:09:43 +00:00 Compare
delliott added a new dependency 2026-01-15 15:09:56 +00:00
tobiasd reviewed 2026-01-15 16:01:15 +00:00
@@ -59,3 +59,3 @@
<p>
Beachte, dass alle FSFE-Events von unserem
<a href="/about/codeofconduct">Verhaltenskodex</a>
<a href="/about/codeofconduct.html">Verhaltenskodex</a>
Owner

The .html is not needed for the link to work as apache2 will resolve it correctly to the matching HTML file.

The `.html` is not needed for the link to work as apache2 will resolve it correctly to the matching HTML file.
Author
Member

I see, I was just making the files match. Four our internal postfixing to link directly it must end in .html. If apache handles it it does not matter though I guess.

I see, I was just making the files match. Four our internal postfixing to link directly it must end in .html. If apache handles it it does not matter though I guess.
Author
Member

I think I will keep the .html ending: I think I prefer to rely on our build system to sort out links than our webserver, as ideally we would eventually be webserver agnostic.

I think I will keep the .html ending: I think I prefer to rely on our build system to sort out links than our webserver, as ideally we would eventually be webserver agnostic.
delliott marked this conversation as resolved
tobiasd reviewed 2026-01-15 16:03:07 +00:00
@@ -7,3 +7,3 @@
</head>
<body class="toplevel article ltr">
<body class="toplevel article">
Owner

The ltr class will prevent the wrong display in rtl languages that have no events.XX.xhtml page and fall back to the English version.

The `ltr` class will prevent the wrong display in `rtl` languages that have no `events.XX.xhtml` page and fall back to the English version.
Author
Member

So should we always be fixing every body with ltr? I would have thought this a problem for css/language based xsl?

So should we always be fixing every body with `ltr`? I would have thought this a problem for css/language based xsl?
Author
Member

I have whitelisted the body for the diff checks, and returned the ltr class.

I have whitelisted the body for the diff checks, and returned the ltr class.
delliott marked this conversation as resolved
mk declined to review 2026-01-16 05:33:43 +00:00
Owner

(I removed myself as technical reviewer.)

(I removed myself as technical reviewer.)
delliott force-pushed fix/events-wiki from 19f3e65716 to daae5f1613 2026-01-25 10:21:12 +00:00 Compare
delliott requested review from tobiasd 2026-01-25 10:26:31 +00:00
tobiasd merged commit 0d99c0f164 into master 2026-01-27 10:32:28 +00:00
tobiasd deleted branch fix/events-wiki 2026-01-27 10:32:29 +00:00
tobiasd referenced this issue from a commit 2026-01-27 10:32:30 +00:00
Sign in to join this conversation.