Description
Details
Subject | Repo | Branch | Lines +/- | |
---|---|---|---|---|
Rephrase notifications | mediawiki/extensions/ContentTranslation | master | +28 -8 |
Related Objects
Event Timeline
It seems as if every other notification wraps everything in a link, whereas this message is only partially linked.
We should clear up the API with T125682. From the UI it is not clear if creating partially linked messages should be allowed.
So they all have one target which means we should change the API to only allow plain text to be passed, not HTML, or at least specify that <a> shouldn't be used.
Yes, we try to standardize them to
- contain no links
- have minimum formatting, we are defining what should be bold or quoted here: T121724: Make page titles bold in notifications
- have one primary link (we make it wrap the entire notification in the fly-out and surface it as the first action on Special:Notifications)
- have 0 to many secondary actions
Yes, restricting the markup that can be returned is certainly an option once the standards are solid and the extensions are migrated.
@Esanders I'm re-purposing your ticket to update the CX notifications, I hope you don't mind.
This sounds like a fairly simple thing to do, but I know relatively little about the internals of Notifications. Can anybody please explain what needs to be done? Does the actual sentence have to change? Or only the part that is linked? Thanks. Also CCing @Pginer-WMF who knows both CX and Notifications, and can provide input if needed.
The copy for many notifications was reviewed to make it more consistent. Regarding Content Translation, the final text is as follows (note it won't contain in-line links in the text):
For Milestone notifications:
- Congratulations on your first translation! Check your contributions and find more pages to translate.
- This will link to Special:MyContributions.
- Congratulations for your tenth translation! You are becoming an expert translator. Keep up the good work!
- This will link to Special:ContentTranslation.
- You just made 100 translations! That is a remarkable achievement. Share the word to other editors.
- This will link to Special:ContentTranslation.
For notification about suggestions:
- Based on your previous translation of '''<last translated page title>''', new suggestions are now available.
- This will link to Special:ContentTranslation (with the parameter that makes it show the suggestions view).
Thanks to @Amire80 - checked in production - the specs are correctly implemented.
The issue with not being able to use Special:ContentTranslation in betalabs was filed as T129284: Content Translation fails to load (and publish) articles at en.wikipedia.beta.wmflabs.org.