Page MenuHomePhabricator

Tpt (Tpt)
User

Projects (13)

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Thursday

  • Clear sailing ahead.

User Details

User Since
Oct 7 2014, 5:56 AM (493 w, 5 h)
Availability
Available
IRC Nick
Tpt
LDAP User
Tpt
MediaWiki User
Tpt [ Global Accounts ]

Recent Activity

Sun, Mar 17

Tpt added a comment to T319965: Migrate phetools from Toolforge GridEngine to Toolforge Kubernetes.
I wasn't able to get one line per wiki, but https://wsstats.toolforge.org/stats/all/alltime exists now which gives a overview of all wikisources.
Sun, Mar 17, 9:08 AM · Tools, All-and-every-Wikisource, Grid-Engine-to-K8s-Migration

Fri, Mar 15

Tpt added a comment to T319965: Migrate phetools from Toolforge GridEngine to Toolforge Kubernetes.

@Soda Amazing! Thank you! A user request from French Wikisource: Would it be possible to add to the website plots for all the wikisources (a line per wiki)? And the data in a table format instead of charts (I guess just allowing to download a csv is fine)?

Fri, Mar 15, 9:20 AM · Tools, All-and-every-Wikisource, Grid-Engine-to-K8s-Migration

Jan 18 2024

Tpt closed T353954: Wikisource OPDS catalog not loading as Resolved.
Jan 18 2024, 4:36 PM · Community-Tech, WS Export

Jan 5 2024

Tpt added a comment to T354267: pr_quality bar should not use <table> for its layout.

I have uploaded a change that converts the <table> to a <ul>. @Poslovitch is it better? If you prefer I can switch to a <div> instead: https://gerrit.wikimedia.org/r/987944

Jan 5 2024, 8:36 AM · Patch-For-Review, Accessibility, ProofreadPage

Dec 23 2023

Tpt added a comment to T353954: Wikisource OPDS catalog not loading.

@Samwilson Thank you so much for investigating this problem!

Dec 23 2023, 5:17 PM · Community-Tech, WS Export

Dec 22 2023

Tpt updated subscribers of T353954: Wikisource OPDS catalog not loading.

This is strange. The file is present on the server but not served anymore by Apache for I dont know what reason.

Dec 22 2023, 7:11 PM · Community-Tech, WS Export

Dec 18 2023

Tpt added a comment to T345406: Add job queue system for generating ebooks.

The controller loop that waits for the book to be generated. Maybe it should do something better.

Dec 18 2023, 8:30 PM · WS Export

Dec 17 2023

Tpt added a comment to T345406: Add job queue system for generating ebooks.

@Samwilson if you think you won't manage to do it in the next few months, I think I can take some time to work on it.

Dec 17 2023, 5:49 PM · WS Export

Dec 3 2023

Tpt added a comment to T352603: Gadget-PrimarySources throws JS error: Uncaught TypeError: async.parallel is not a function.

+1 to @Tacsipacsi. It's probably the best way to go. Would someone have time to do it? Help would very much welcomed

Dec 3 2023, 1:06 PM · Wikidata, Regression, Wikidata-primary-sources

Aug 19 2023

Tpt added a comment to T344544: Archive the DoubleWiki extension.

I am not actively maintaining the extension. I never took the time to try to rewrite it properly. I am fine with undeploying it. Some consultation of the users (frwikisource...) might be nice to avoid a backslash.

Aug 19 2023, 10:36 AM · User-notice, Community-consensus-needed, All-and-every-Wikisource, MediaWiki-extensions-DoubleWiki, translatewiki.net, Wikimedia-GitHub, Diffusion-Repository-Administrators, Projects-Cleanup

Mar 18 2023

Tpt added a comment to T332450: Upgrade ws-export VMs to Bullseye.

Idea: it might be nice to use Terraform for the migration to ease future migrations (Bookworm...)

Mar 18 2023, 7:55 AM · Community-Tech, WS Export
Tpt added a parent task for T332450: Upgrade ws-export VMs to Bullseye: T254937: Wikisource Ebooks: Investigate cover page bug [8h].
Mar 18 2023, 7:54 AM · Community-Tech, WS Export
Tpt added a subtask for T254937: Wikisource Ebooks: Investigate cover page bug [8h]: T332450: Upgrade ws-export VMs to Bullseye.
Mar 18 2023, 7:54 AM · WS Export

Mar 17 2023

Tpt updated the task description for T332450: Upgrade ws-export VMs to Bullseye.
Mar 17 2023, 8:12 PM · Community-Tech, WS Export
Tpt created T332450: Upgrade ws-export VMs to Bullseye.
Mar 17 2023, 8:12 PM · Community-Tech, WS Export

Feb 28 2023

Tpt committed R2221:9671988e6037: Downgrades phpunit (authored by Tpt).
Downgrades phpunit
Feb 28 2023, 12:56 AM
Tpt committed R2221:9025b285c080: ws2wd: strips HTML tags (authored by Tpt).
ws2wd: strips HTML tags
Feb 28 2023, 12:56 AM
Tpt committed R2221:4cbc6b96fc60: Removes debug print (authored by Tpt).
Removes debug print
Feb 28 2023, 12:56 AM
Tpt committed R2221:3493cff5905b: Upgrades dependencies (authored by Tpt).
Upgrades dependencies
Feb 28 2023, 12:56 AM

Feb 27 2023

Tpt closed T270655: simplewd tool is in a crash loop -- please fix as Invalid.

simplewd has been deleted

Feb 27 2023, 2:35 PM · Tools
Tpt placed T268932: Add Bengali currency/hexadecimal numerals to pagelist up for grabs.
Feb 27 2023, 2:33 PM · User-notice-archive, MW-1.41-notes (1.41.0-wmf.25; 2023-09-05), Wikimania-Hackathon-2023, MW-1.36-notes (1.36.0-wmf.28; 2021-01-26), Bengali-Sites, I18n, ProofreadPage

Feb 16 2023

Tpt added a comment to T329432: Add a tracking category when the section name given to the #lst parser function does not exist.

@Billinghurst @Xover Sorry for that. I have just pushed a fix for review: https://gerrit.wikimedia.org/r/c/mediawiki/extensions/LabeledSectionTransclusion/+/889867/

Feb 16 2023, 9:22 PM · MW-1.40-notes (1.40.0-wmf.24; 2023-02-20), MediaWiki-extensions-LabeledSectionTransclusion

Feb 11 2023

Tpt created T329432: Add a tracking category when the section name given to the #lst parser function does not exist.
Feb 11 2023, 8:57 PM · MW-1.40-notes (1.40.0-wmf.24; 2023-02-20), MediaWiki-extensions-LabeledSectionTransclusion

Jan 23 2023

Tpt added a comment to T326952: CVE-2023-37253: ProofreadPage leaks suppressed user via the API and config variables.

@Soda Thank you for reaching out with this problem and moving forward with it.

Jan 23 2023, 4:35 PM · Patch-For-Review, Vuln-Infoleak, ProofreadPage, Security, Security-Team

Nov 1 2022

Tpt updated subscribers of T254937: Wikisource Ebooks: Investigate cover page bug [8h].

This bug seems to have been fixed upstream in ImageMagick https://github.com/ImageMagick/ImageMagick/issues/2070
We should upgrade it in WSexport servers

Nov 1 2022, 9:41 AM · WS Export

Oct 31 2022

Tpt added a comment to T93397: Modifying an Index page update Special:IndexPages with the previous revision of the Index page, not with current one.

@Zdzislaw I have made some refactoring to properly cleanup the way the index quality stats are updated. It is likely it has fixed this bug in the process.

Oct 31 2022, 8:18 PM · ProofreadPage

Oct 27 2022

Tpt removed a project from T319258: rest api incorrectly places references: Parsoid.

After some investigation, it seems this bug has been introduced by the migration to Parsoid. It's likely that Parsoid fully parses wikitext from custom MW tags like <pages>, including references, before including them in Parsoid output. Adding native support of <pages> to Parsoid might fix this problem (or not, not sure...).

Oct 27 2022, 12:38 PM · ProofreadPage, WS Export, Regression, All-and-every-Wikisource
Tpt edited projects for T319258: rest api incorrectly places references, added: Parsoid; removed RESTBase-API.
Oct 27 2022, 12:29 PM · ProofreadPage, WS Export, Regression, All-and-every-Wikisource

Oct 15 2022

Tpt added a comment to T53980: Source tab not showing up in the Translation namespace.

It sounds like one of the main issues with doing that might be on talk pages, where multiple different Index pages might be linked (for examples etc.), but actually I think this is highlighting an existing problem with how we handle linking to multiple sources. For example, a top-level page for a work might want to display the tables of contents from multiple volumes with something like this (where each volume has its ToC on page 5):

Oct 15 2022, 10:25 AM · Bengali-Sites, MW-1.40-notes (1.40.0-wmf.6; 2022-10-17), Thai-Sites, ProofreadPage

Sep 22 2022

Tpt closed T318266: The proofreading progress indicator has a 0 size width on 1.40.0-wmf.2 as Resolved.

Fixed on Wikisources

Sep 22 2022, 8:26 PM · MW-1.40-notes (1.40.0-wmf.2; 2022-09-19), All-and-every-Wikisource, ProofreadPage

Sep 21 2022

Tpt added a comment to T318266: The proofreading progress indicator has a 0 size width on 1.40.0-wmf.2.

Quick fix to add to Common.css:

.pr_quality {
	width: 100px;
}
Sep 21 2022, 7:35 PM · MW-1.40-notes (1.40.0-wmf.2; 2022-09-19), All-and-every-Wikisource, ProofreadPage
Tpt triaged T318266: The proofreading progress indicator has a 0 size width on 1.40.0-wmf.2 as High priority.
Sep 21 2022, 7:25 PM · MW-1.40-notes (1.40.0-wmf.2; 2022-09-19), All-and-every-Wikisource, ProofreadPage
Tpt created T318266: The proofreading progress indicator has a 0 size width on 1.40.0-wmf.2.
Sep 21 2022, 7:24 PM · MW-1.40-notes (1.40.0-wmf.2; 2022-09-19), All-and-every-Wikisource, ProofreadPage

Jul 2 2022

Tpt created T311901: Special:PagesWithoutScans shows only one entry repeated 5000 times.
Jul 2 2022, 9:58 AM · MW-1.39-notes (1.39.0-wmf.25; 2022-08-15), ProofreadPage

May 29 2022

Tpt added a comment to T309451: Live preview shows incorrect proofread status and headers.

Indeed, this bug is related to c1f80efe05349cc39cb82dc9347b0a7699b11dfd (T304303): the live preview feature relies on the JS API allowing to fetch the current content of the editing area. To fix T183950 we dropped the hack inside of ProofreadPage that provides not only the page body but also the proofreading level and the page header/footer as page content. So, after this change, only the page body is fed to the live preview feature, hence the bug. A possible solution might be to add back the "complete page content" hack inside of ProofreadPage but some signficant work would be required to make features like find/replace work with this hack.

May 29 2022, 10:26 AM · MediaWiki-Page-editing, ProofreadPage

May 26 2022

Tpt added a comment to T128173: Represent editions as interwiki links on Wikisource.

I have written an implementation draft: https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Wikisource/+/793867

May 26 2022, 2:43 PM · Patch-For-Review, MW-1.35-notes (1.35.0-wmf.36; 2020-06-09), All-and-every-Wikisource, Wikidata

May 9 2022

Samwilson awarded T307956: IA-upload: Drop phetools dependency a Like token.
May 9 2022, 11:22 PM · IA Upload
Tpt created T307956: IA-upload: Drop phetools dependency.
May 9 2022, 7:16 PM · IA Upload

Mar 27 2022

Tpt created T304795: Drop the ProofreadPageUseStatusChangeTags option.
Mar 27 2022, 1:59 PM · Patch-For-Review, Wikimedia-Site-requests, ProofreadPage

Feb 14 2022

Tpt added a comment to T301655: HTML entity error (unencoded ampersand) in OPDS export for Wikisource.

A fix has just been merged https://github.com/wikimedia/ws-export/pull/397 with a nicer follow up https://github.com/wikimedia/ws-export/pull/398

Feb 14 2022, 8:44 AM · Community-Tech, WS Export

Jan 31 2022

Tpt updated the task description for T289760: Evaluate Oxigraph as alternative to Blazegraph.
Jan 31 2022, 7:28 AM · Wikidata, Wikidata-Query-Service

Jan 13 2022

Tpt added a comment to T299124: ProofreadPage frontend makes a request to the page before and after in every page view.

rel=prefetch has been added targetting the previous and next pages to speed up bulk proofreading according to T230689

Jan 13 2022, 10:49 AM · MW-1.40-notes (1.40.0-wmf.7; 2022-10-24), Performance-Team (Radar), ProofreadPage

Jan 7 2022

Tpt added a comment to T298694: ProofreadPage: zoom/pan not working in side-by-side editing mode.

@Xover Sadly there is not hope for it to be pushed today except if someone is managing to follow the emergency deployment process: https://wikitech.wikimedia.org/wiki/Deployments/Emergencies

Jan 7 2022, 11:18 AM · MW-1.38-notes (1.38.0-wmf.16; 2022-01-03), ProofreadPage
Tpt triaged T298694: ProofreadPage: zoom/pan not working in side-by-side editing mode as High priority.

High priority: breaks a part of the basic Wikisource proofreading workflow.

Jan 7 2022, 8:20 AM · MW-1.38-notes (1.38.0-wmf.16; 2022-01-03), ProofreadPage

Jan 6 2022

Tpt added a comment to T298694: ProofreadPage: zoom/pan not working in side-by-side editing mode.

@Inductiveload Thank you!

Jan 6 2022, 5:41 PM · MW-1.38-notes (1.38.0-wmf.16; 2022-01-03), ProofreadPage
Tpt added a comment to T298694: ProofreadPage: zoom/pan not working in side-by-side editing mode.

I wrote a change that should fix the bug https://gerrit.wikimedia.org/r/c/mediawiki/extensions/ProofreadPage/+/751946/

Jan 6 2022, 4:11 PM · MW-1.38-notes (1.38.0-wmf.16; 2022-01-03), ProofreadPage

Jan 4 2022

Tpt added a comment to T292365: IA-Upload does not take care of disk space when accepting jobs.

The bug happend again on the new disk with the "exhibitorsherald2*" files. I removed them to put the tool back on. We should maybe prevent files >1GB to be downloaded and processed.

Jan 4 2022, 8:04 PM · IA Upload

Dec 8 2021

Soda awarded T291082: Grant Inductiveload +2 right on ProofreadPage a Like token.
Dec 8 2021, 4:28 PM · ProofreadPage, Gerrit-Privilege-Requests

Dec 1 2021

Tpt added a comment to T285610: Pages corresponding to tables of contents' second links are not provided the first link as the "prev" parameter for headers generated with <pages>.

@Xover Thank you so much for having found this issue. It was under my nose all this time.

Dec 1 2021, 3:51 PM · MW-1.38-notes (1.38.0-wmf.16; 2022-01-03), ProofreadPage

Nov 24 2021

Tpt closed T217488: Move ProofreadPage class into ProofreadPage namespace as Resolved.
Nov 24 2021, 4:28 PM · ProofreadPage
Tpt added a comment to T285610: Pages corresponding to tables of contents' second links are not provided the first link as the "prev" parameter for headers generated with <pages>.

@Mahir256 The change I wrote is in fact buggy. It's looking for a field named "toc" and not a field which data field is "toc". I'm writing a fix right now. Sorry for that.

Nov 24 2021, 2:05 PM · MW-1.38-notes (1.38.0-wmf.16; 2022-01-03), ProofreadPage

Nov 23 2021

Tpt added a comment to T296325: Investigate Page namespace 1.5x/2x image without zooming in .

@Soda do you have a HiDPI screen? If it's the case, I believe modern browsers will download and use the bigger image that is provided by the srcset attribute before the initialization of Openseadragon. Then Openseadragon is initialized and use the regular size image. It's what happens with my 4K screen configured with 2x scaling.

Nov 23 2021, 7:39 PM · ProofreadPage

Nov 21 2021

Tpt added a comment to T268932: Add Bengali currency/hexadecimal numerals to pagelist.

Sorry, I made a confusion between two different numerical systems: the ICU "beng" numerals are defined as the 10 digits "০১২৩৪৫৬৭৮৯" and, so, are not using base 16 as the currency numerals requested by this task. The base 10 numerical system is properly implemented and available now, the base 16 numerals are not yet availlable and not provided by ICU.

Nov 21 2021, 7:51 PM · User-notice-archive, MW-1.41-notes (1.41.0-wmf.25; 2023-09-05), Wikimania-Hackathon-2023, MW-1.36-notes (1.36.0-wmf.28; 2021-01-26), Bengali-Sites, I18n, ProofreadPage
Tpt added a comment to T268932: Add Bengali currency/hexadecimal numerals to pagelist.

@Inductiveload I believe @Bodhisattwa says that the "beng" option is not visible in the pagelist editing widget.

Nov 21 2021, 7:36 PM · User-notice-archive, MW-1.41-notes (1.41.0-wmf.25; 2023-09-05), Wikimania-Hackathon-2023, MW-1.36-notes (1.36.0-wmf.28; 2021-01-26), Bengali-Sites, I18n, ProofreadPage

Nov 19 2021

Tpt added a comment to T289760: Evaluate Oxigraph as alternative to Blazegraph.

@Thadguidry @BenAtOlive It's a great idea to move the conversation somewhere else. Oxigraph has a discussion space and a Gitter channel.

Nov 19 2021, 8:05 PM · Wikidata, Wikidata-Query-Service
Tpt added a comment to T289760: Evaluate Oxigraph as alternative to Blazegraph.

@BenAtOlive If I understand correctly, you are asking if Oxigraph can be used to build a distributed SPARQL database with reader nodes and writer nodes? Its definitely possible to reuse Oxigraph components for that. For example I am currently moving the SPARQL parser out of Oxigraph and I plan to add general SPARQL query optimizations to it. However, building a query evaluator against a distributed storage where data are not in the same machine as the one executing (or at least scheduling) the query is a quite different task as building a query executor against a single node database. Access latencies are easily an order of magnitude higher with a remote store and it changes quite a lot of tradeoffs. So, yes for making the part of Oxigraph that are easy to reuse reusable but sadly I am not sure that naively spliting the query evaluator out of Oxigraph would provide a good query evaluator on top of a distributed storage.

Nov 19 2021, 7:44 PM · Wikidata, Wikidata-Query-Service

Nov 18 2021

Tpt closed T285610: Pages corresponding to tables of contents' second links are not provided the first link as the "prev" parameter for headers generated with <pages> as Resolved.

The change fixing this issue is now merged.

Nov 18 2021, 1:54 PM · MW-1.38-notes (1.38.0-wmf.16; 2022-01-03), ProofreadPage

Nov 16 2021

Tpt added projects to T295771: Increase $wgMaximumMovedPages for eo.wikisource: Wikimedia-Site-requests, All-and-every-Wikisource.
Nov 16 2021, 1:55 PM · MoveComms-Support, Wikimedia-Site-requests

Nov 15 2021

Tpt added a comment to T295275: Dedicated section on Wikidata Item and Property pages for classifying Properties.

It's a great idea! I just believe P2860 (cites work) should be removed from the list. It's definitely not a classifying property.

Nov 15 2021, 12:34 PM · Wikidata, Wikidata-Campsite

Oct 3 2021

Tpt triaged T292365: IA-Upload does not take care of disk space when accepting jobs as High priority.
Oct 3 2021, 12:40 PM · IA Upload
Tpt closed T292361: ia-upload server needs a prod as Resolved.

It is now fixed. There was some broken job data in the file system because of the full disk problem. I have opened T292365 to prevent it to happen again.

Oct 3 2021, 12:40 PM · IA Upload
Tpt created T292365: IA-Upload does not take care of disk space when accepting jobs.
Oct 3 2021, 12:40 PM · IA Upload
Tpt added a comment to T292361: ia-upload server needs a prod.

It's because the disk is currently full. I have removed a few huge books from the job queue and rebooted the server. However it does not seems to be enough to bring the website back.

Oct 3 2021, 12:25 PM · IA Upload

Oct 2 2021

Tpt committed R2221:88e916108c6c: Upgrades dependencies (authored by Tpt).
Upgrades dependencies
Oct 2 2021, 2:35 PM

Sep 17 2021

Tpt added a comment to T114318: Sometimes the anchor links generated by an Index: page's <pagelist /> do not have the expected class attribute and value set.

@Mpaa @Dvorapa @Dalba Sorry for distrubing you. Is this problem fixed now?

Sep 17 2021, 7:54 PM · TestMe, Pywikibot-tests, ProofreadPage, Pywikibot
Tpt added a comment to T290578: Use the change_tag table to store the proofreading quality level.

@Ladsgroup Thank you! So what would you think about MCR for storing the "main" data and some change tags as "secondary" data for queryability and recent changes filtering? This way we would get the best of both worlds.

Sep 17 2021, 2:18 PM · ProofreadPage

Sep 15 2021

Tpt added a comment to T290578: Use the change_tag table to store the proofreading quality level.

Thank you @Tgr and @Ladsgroup for your feedbacks!

Sep 15 2021, 9:02 PM · ProofreadPage
Tpt created T291082: Grant Inductiveload +2 right on ProofreadPage.
Sep 15 2021, 1:11 PM · ProofreadPage, Gerrit-Privilege-Requests

Sep 14 2021

Tpt updated subscribers of T290578: Use the change_tag table to store the proofreading quality level.

@Tgr @Legoktm Sorry for bothering you. I believe you where at some point involved in with change tags system. Do you think our plan seems sensible?

Sep 14 2021, 3:02 PM · ProofreadPage

Sep 13 2021

Tpt added a comment to T289140: ProofreadPage: Enable change-tag status system on Wikisources.

@Inductiveload Great question! We might start now. Maybe with oldwikisource?

Sep 13 2021, 6:14 PM · Patch-For-Review, ProofreadPage

Sep 8 2021

Tpt created T290578: Use the change_tag table to store the proofreading quality level.
Sep 8 2021, 3:05 PM · ProofreadPage

Sep 2 2021

Tpt added a comment to T290243: Index pages no longer show elements on the "edit" page between "Pages" and "Categories" (Hebrew Wikisource).

This still doesn't explain how the fields were there earlier, as the current text shows (which cannot be edited).

Sep 2 2021, 1:19 PM · ProofreadPage
Tpt added a comment to T290243: Index pages no longer show elements on the "edit" page between "Pages" and "Categories" (Hebrew Wikisource).

@Aklapper Thank you! Most of Wikisource now use that is an improved version of Mediawiki:Proofreadpage_index_attributes. MediaWiki:Proofreadpage_index_data_config is defined in the Hebrew Wikisource.

Sep 2 2021, 1:14 PM · ProofreadPage

Aug 31 2021

Tpt added a comment to T289760: Evaluate Oxigraph as alternative to Blazegraph.

@Thadguidry Great! Thank you!

Aug 31 2021, 2:47 PM · Wikidata, Wikidata-Query-Service
Tpt added a comment to T289760: Evaluate Oxigraph as alternative to Blazegraph.

@Thadguidry Hi! It's a great idea to improve the roadmap. Thank you for pushing me to do that! I have updated the GitHub issues and milestones and added a link to them in the README:
Each milestone should now have a short description and a set of linked issues. Most of the issues have been connected to a milestones.
I am a bit afraid of adding a ROADMAP file to the repository because it is yet another source of truth to maintain.
To also keep a single source of truth I made the choice to not expand the milestone description too much and keep the detailed discussions inside of issues. Here is [one about the storage backend and Sled]( https://github.com/oxigraph/oxigraph
/milestones?direction=desc&sort=completeness&state=open). However, the milestone+connected issues descriptions should allow to make sense of the roadmap. Feel free to ping me if you believe it's not clear enough.

Aug 31 2021, 6:57 AM · Wikidata, Wikidata-Query-Service

Aug 26 2021

Tpt added a comment to T289760: Evaluate Oxigraph as alternative to Blazegraph.

Thank you @So9q for opening this task!

Aug 26 2021, 3:57 PM · Wikidata, Wikidata-Query-Service

Aug 25 2021

Tpt added a comment to T289728: ProofreadPage\PageNumberNotFoundException: Page:[page] provides invalid page number .

I'm working on a fix. Should be ready soon. Sorry for the troubles.

Aug 25 2021, 8:18 PM · MW-1.37-notes (1.37.0-wmf.20; 2021-08-23), Vector (legacy skin) (Tracking), Platform Engineering, ProofreadPage, User-brennen, Wikimedia-production-error

Aug 12 2021

Tpt closed T288408: First paragraph should not have text-indent:0 in Page: namespace if inside a parent container has already set this to something else. as Resolved.

Fixed and deployed.

Aug 12 2021, 8:43 AM · MW-1.37-notes (1.37.0-wmf.18; 2021-08-09), ProofreadPage

Aug 7 2021

Tpt added a comment to T285610: Pages corresponding to tables of contents' second links are not provided the first link as the "prev" parameter for headers generated with <pages>.

Sorry, I should not have closed this issue. The change that should fix this problem is not merged yet: https://gerrit.wikimedia.org/r/c/mediawiki/extensions/ProofreadPage/+/704564
So, it makes sense that the problem does not seem solved on Wikisource.

Aug 7 2021, 7:43 AM · MW-1.38-notes (1.38.0-wmf.16; 2022-01-03), ProofreadPage

Aug 5 2021

Tpt added a comment to T288191: substituting {{#tag:ref}} tags and templates; and pipe tricks fail in Page namespace.

How widespread is the impact of this bug remaining unfixed? If it's a nitch use and we can fix it with a backport in the short term then I wouldn't roll back. On the other hand, if this is causing some fallout elsewhere or triggering major production breakage then definitely roll back.

Aug 5 2021, 8:11 PM · MW-1.37-notes (1.37.0-wmf.17; 2021-08-02), MediaWiki-ContentHandler, Platform Team Workboards (MW Expedition), Regression, ProofreadPage

Aug 4 2021

Tpt closed T285610: Pages corresponding to tables of contents' second links are not provided the first link as the "prev" parameter for headers generated with <pages> as Resolved.

Yes. To make it work properly "data": "toc" should be added to the relevant field in Mediawiki:Proofreadpage_index_data_config.

Aug 4 2021, 4:43 PM · MW-1.38-notes (1.38.0-wmf.16; 2022-01-03), ProofreadPage

Jul 31 2021

Tpt renamed T287809: phpcs should validate @return phpdoc annotations of abstract methods from phpcs should warn against common type variants like "boolean" or "integer" in PHPdoc comments to phpcs should validate @return phpdoc annotations of abstract methods.
Jul 31 2021, 7:38 AM · MediaWiki-Codesniffer
Tpt added a comment to T287809: phpcs should validate @return phpdoc annotations of abstract methods.

@DannyS712 Thank you! I have understood why e.g. https://phabricator.wikimedia.org/diffusion/EPRP/browse/master/includes/Pagination/Pagination.php$25 is not flagged by these sniffs. It's because abstract methods are not covered by the lint: https://github.com/wikimedia/mediawiki-tools-codesniffer/blob/5f813f727ee47d5c90981b44e82f0c5a0263e933/MediaWiki/Sniffs/Commenting/FunctionCommentSniff.php#L198

Jul 31 2021, 7:33 AM · MediaWiki-Codesniffer
Tpt created T287809: phpcs should validate @return phpdoc annotations of abstract methods.
Jul 31 2021, 7:18 AM · MediaWiki-Codesniffer

Jul 19 2021

Tpt added a comment to T230689: Preload the next page's image on proofreading view.

Thank you @Ankry, @Xover @Inductiveload! So I guess rel="preload" seems the way to go for the next page image. What about using rel="prefetch" for the previous one? This way, it gets a good chance to be loaded too but with less priority. What do you think about it?

Jul 19 2021, 5:18 PM · Community-Tech (CommTech-Sprint-7), MW-1.37-notes (1.37.0-wmf.17; 2021-08-02), Patch-For-Review, MW-1.36-notes (1.36.0-wmf.20; 2020-12-01), ProofreadPage

Jul 17 2021

Tpt added a comment to T230689: Preload the next page's image on proofreading view.

@Ankry Thank you! So, I guess that the main goal is to make sure that the image thumbnail is already generated when a Page: page is displayed.

Jul 17 2021, 4:28 PM · Community-Tech (CommTech-Sprint-7), MW-1.37-notes (1.37.0-wmf.17; 2021-08-02), Patch-For-Review, MW-1.36-notes (1.36.0-wmf.20; 2020-12-01), ProofreadPage
Tpt added a comment to T230689: Preload the next page's image on proofreading view.

If I remind correctly preload is to ask the browser to aggressively fetch a content for the current page rendering and prefetch is to ask the browser to load the content for future use in the navigation.

Jul 17 2021, 7:41 AM · Community-Tech (CommTech-Sprint-7), MW-1.37-notes (1.37.0-wmf.17; 2021-08-02), Patch-For-Review, MW-1.36-notes (1.36.0-wmf.20; 2020-12-01), ProofreadPage

Jul 13 2021

Tpt created T286582: Duplicated links to Page: pages should be allowed in Index: pages relying on a PDF/DjVu file.
Jul 13 2021, 6:01 PM · Patch-Needs-Improvement, All-and-every-Wikisource, ProofreadPage

Jun 30 2021

Tpt added a comment to T285610: Pages corresponding to tables of contents' second links are not provided the first link as the "prev" parameter for headers generated with <pages>.

But aiui the granularity would be per-project and not per-index. How would a project start experimenting with this or avoid the need for a big bang migration? Use two toc fields in the index, one for the traditional way and one for a "Wikidata-based toc"? That would necessitate PRP differentiate between "present" and "non-empty" for the index data field tagged with "data": "toc", I think. That sounds a little more complicated, but is perhaps not too much so?

Jun 30 2021, 8:04 PM · MW-1.38-notes (1.38.0-wmf.16; 2022-01-03), ProofreadPage

Jun 29 2021

Tpt added a comment to T285610: Pages corresponding to tables of contents' second links are not provided the first link as the "prev" parameter for headers generated with <pages>.

Thank you @Mahir256 and @Xover for this discussion.

Jun 29 2021, 8:08 PM · MW-1.38-notes (1.38.0-wmf.16; 2022-01-03), ProofreadPage

Jun 8 2021

Tpt added a comment to T274350: ProofreadPage source tab not shown in the Minerva skin.

The "Source" tab is currently inserted using javascript ("modules/article/ext.proofreadpage.article.js"). It might be possible to use the same trick for Minerva. But migrating everything to PHP would be nicer.

Jun 8 2021, 7:14 AM · MinervaNeue (Tracking), Mobile, ProofreadPage
Tpt added a comment to T284504: ProofreadPages: Allow <pages/> tag on index pages.

@Tpt Is there a technical reason for this?

Jun 8 2021, 4:46 AM · ProofreadPage

May 22 2021

Tpt closed T268932: Add Bengali currency/hexadecimal numerals to pagelist as Resolved.

Done and deployed

May 22 2021, 1:43 PM · User-notice-archive, MW-1.41-notes (1.41.0-wmf.25; 2023-09-05), Wikimania-Hackathon-2023, MW-1.36-notes (1.36.0-wmf.28; 2021-01-26), Bengali-Sites, I18n, ProofreadPage
Tpt added a comment to T279255: GSoC 2021 Proposal: Add zoom and pan to the Wikisource Pagelist Widget.

Sorry for stepping in. A maybe relevant task when designing a new zooming system: T43614

May 22 2021, 12:53 PM · Patch-For-Review, ProofreadPage, Google-Summer-of-Code (2021)

Mar 27 2021

Tpt added a comment to T93608: Switch ProofreadPage to use indicator tags.

@He7d3r @GOIII are you fine with me closing this task as "declined" or do you want to move this forward, maybe by getting feedback from the Wikisource community?

Mar 27 2021, 12:35 PM · ProofreadPage
Tpt added a comment to T93608: Switch ProofreadPage to use indicator tags.

After thinking a little bit (and asking my wife who does not contribute to Wikisource), I have a feeling using an indicator is maybe not highlighting enough the proofreading level information: it's the first thing people are often looking for if they go to a page: page. But I am not an UX design expert so maybe an indicator is good enough.

Mar 27 2021, 8:03 AM · ProofreadPage
Tpt added a comment to T93608: Switch ProofreadPage to use indicator tags.

Thank you for pushing this. I am a bit afraid it might create a backlash with some contributors feeling that the proofreading level is not highlighted enough. It might be nice to start a discussion on the biggest Wikisource communities. A first step is maybe to move the quality level from the <pages> tag and in the Index namespaces to an indicator.

Mar 27 2021, 7:54 AM · ProofreadPage
Tpt added a comment to T53980: Source tab not showing up in the Translation namespace.

This is presumably very nearly T239033, since if the extension can add a "Source" link for a mainspace page user of <pages/>, it can do it in any other namespace?

Mar 27 2021, 7:26 AM · Bengali-Sites, MW-1.40-notes (1.40.0-wmf.6; 2022-10-17), Thai-Sites, ProofreadPage
Tpt added a comment to T239033: Make ProofreadPage insert the "Source" link server side in the main namespace.

@Inductiveload Yes, I guess it should be also done in onSkinTemplateNavigation. The tricky part is that the "source" link is not known from the current page title but from the parser output, and onSkinTemplateNavigation do not have access to the parser output. It's why I have not done the migration to server side yet. There is maybe a good solution, I have not taken much time to look for an alternative yet.

Mar 27 2021, 7:24 AM · ProofreadPage

Mar 26 2021

Tpt moved T278564: Codesniffer should warn if output functions like print are used from Untriaged to Proposed rule changes on the MediaWiki-Codesniffer board.
Mar 26 2021, 4:33 PM · phan, MediaWiki-Codesniffer