Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Underpinnings/preludes/fixes to proper insertion of images. #4074

Merged
merged 3 commits into from
Jul 8, 2023

Conversation

dbrant
Copy link
Member

@dbrant dbrant commented Jul 7, 2023

[note that this PR merges onto another open PR]

This makes a number of enhancements, preparations, and fixes to our current workflow of adding images in the Editing activity.

The biggest flaw in our current way of inserting images: it's not localized!
Meaning, we currently insert a template that looks like this:
[[File:<filename>|thumb|alt=<alt text>|<caption>]]

This is technically ok, but we use the same template for all language wikis, whereas the File: namespace and the thumb and alt= magic words should be localized to the target language, but they're not.

This PR adds API functionality that fetches a list of magic words from the target wiki, so that this template can be properly localized. This also makes the resulting template a bit more flexible by excluding parameters that are defaults.

@dbrant dbrant added the Minor Minor stuff label Jul 7, 2023
Base automatically changed from intentParams to main July 7, 2023 23:53
@sharvaniharan sharvaniharan merged commit 183849a into main Jul 8, 2023
1 check passed
@sharvaniharan sharvaniharan deleted the imageParamsLocalize branch July 8, 2023 02:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Minor Minor stuff
2 participants