Skip to content

Add report link to mediawiki footer#488

Merged
AndrewKostka merged 1 commit into
mainfrom
report-link-footer
Aug 13, 2025
Merged

Add report link to mediawiki footer#488
AndrewKostka merged 1 commit into
mainfrom
report-link-footer

Conversation

@dati18
Copy link
Copy Markdown
Contributor

@dati18 dati18 commented Aug 12, 2025

Bug: T397906

Copy link
Copy Markdown
Member

@outdooracorn outdooracorn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested this locally and it works. I have one minor nitpick and a few questions for my understanding.

$footerlinks['report'] = Html::element(
'a',
[
'href' => 'https://wikibase.cloud/complaint',
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess there isn't an easy way to tell what platform the wiki is running on (e.g. wikibase.cloud, wikibase.dev, or wbaas.dev)?

@@ -218,6 +218,20 @@
],
];

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's follow the existing pattern in this file and add a comment explaining what this config does and a link to the docs.

Suggested change
// Add 'report illegal content' link
// https://www.mediawiki.org/wiki/Manual:Hooks/SkinAddFooterLinks

'target' => '_blank',
'rel' => 'noopener noreferrer'
],
'Report illegal content'
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The docs use i18n messages. I guess this isn't possible on Cloud currently as it's not hooked up to a translation pipeline?

@AndrewKostka AndrewKostka merged commit 171b450 into main Aug 13, 2025
9 checks passed
@AndrewKostka AndrewKostka deleted the report-link-footer branch August 13, 2025 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants