TheDJ (Derk-Jan Hartman)
Volunteer developer

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Wednesday

  • Clear sailing ahead.

User Details

User Since
Oct 6 2014, 10:32 PM (196 w, 6 d)
Availability
Available
IRC Nick
thedj
LDAP User
TheDJ
MediaWiki User
TheDJ [ Global Accounts ]
  • Open Source dev since 2001
  • Wikipedian since 2005
  • Wikipedia admin since 2008
  • Commons admin 2010-2015
  • MediaWiki dev (+2) since 2009

Uses Safari most of the time (because someone has to)

Recent Activity

Sat, Jul 14

TheDJ added a comment to T198021: Resolve inconsistent numbering of references in text and in bibliography.

Most tasks are neglected, because we have too much stuff to do. Trying to “me first” a task will not change that, just annoy people.

Sat, Jul 14, 7:49 PM · Design, Need-volunteer, Cite

Fri, Jul 13

TheDJ added a comment to T199041: Pswikivoyage content pages counting issues.

So what led you to conclude that you have over 360 content pages then ? Where is that number coming from ?

Fri, Jul 13, 1:02 PM · Wikimedia-maintenance-script-run

Thu, Jul 12

TheDJ added a comment to T199478: Gadgets should use the current skin, instead of the skin preference.

Will need a fix for the api module. Makes me wonder what the implied behavior of isAllowed (and the corresponding api flag) is actually supposed to be. Would be nice if we could track down what the original intent was 'relevant to the user' or 'allowed for the user'. Did the first just get slapped onto the latter ?

Thu, Jul 12, 10:02 PM · Patch-For-Review, Gadgets
TheDJ triaged T199478: Gadgets should use the current skin, instead of the skin preference as Normal priority.
Thu, Jul 12, 9:28 PM · Patch-For-Review, Gadgets
TheDJ updated subscribers of T199478: Gadgets should use the current skin, instead of the skin preference.
Thu, Jul 12, 9:27 PM · Patch-For-Review, Gadgets
TheDJ added a comment to T127147: Gadget definition "targets=mobile" seem to target mobile and desktop users on ur.wp.

It's a simple fix, but it was stalled waiting review. If the issue is still relevant a year and half later and someone wants to review I'm happy to rebase. The main thing I personally wasn't sure of was if my test was any good or made any sense.

Thu, Jul 12, 9:27 PM · Community-Tech (2015-2017), Gadgets
TheDJ created T199478: Gadgets should use the current skin, instead of the skin preference.
Thu, Jul 12, 9:25 PM · Patch-For-Review, Gadgets
TheDJ added a comment to T199150: Audio player is non-functional on first load in Chrome.

See it on Safari too at times. Cause is more likely that they use Javascript to move the fragment, while the player is loading. This probably messes up some logic. They should be using something like https://www.mediawiki.org/wiki/Help:Page_status_indicators instead.

Thu, Jul 12, 11:57 AM · TimedMediaHandler

Wed, Jul 11

TheDJ renamed T199366: Allow InputBox parameters to be defined as XML attributes from Allow InputBox parameters to be defined as XML attributes to Allow InputBox parameters to be defined as HTML attributes.
Wed, Jul 11, 8:27 PM · MediaWiki-extensions-InputBox

Tue, Jul 10

TheDJ added a comment to T198724: Hatnote templates are under first paragraph on cswiki.

Example link: https://cs.wikipedia.org/api/rest_v1/page/mobile-sections-lead/Slezsko

Tue, Jul 10, 10:34 PM · Wikipedia-Android-App-Backlog, Reading-Web-Local-Wiki-Issues, Readers-Web-Backlog (Tracking), MobileFrontend, Mobile
TheDJ added a comment to T199258: Make global preferences global.

You do not need check all wikis, just to have a hardcoded list of extensions that use global preferences.

Tue, Jul 10, 10:26 PM · Community-Tech, MediaWiki-extensions-GlobalPreferences
TheDJ added a comment to T199258: Make global preferences global.

For example, It includes education extension notifications only on wikis that have this extension deployed. [..] This means, for example, that all extensions in the world should be there, in any wiki, so this page will be the same on all wikis.

Tue, Jul 10, 10:16 PM · Community-Tech, MediaWiki-extensions-GlobalPreferences
TheDJ added a comment to T199258: Make global preferences global.

New issues require new tickets. Which is not the same as sub tickets. subtickets are 'a part of' a parent task. Normally, they would be resolved before a parent task ever can be resolved. Here the parent task can be resolved without implementing this ticket.

Tue, Jul 10, 10:06 PM · Community-Tech, MediaWiki-extensions-GlobalPreferences
TheDJ added projects to T199274: Use localstorage for sitenotices instead of cookies: JavaScript, Easy.
Tue, Jul 10, 8:44 PM · Easy, JavaScript, Technical-Debt, DismissableSiteNotice
TheDJ created T199274: Use localstorage for sitenotices instead of cookies.
Tue, Jul 10, 8:38 PM · Easy, JavaScript, Technical-Debt, DismissableSiteNotice

Mon, Jul 9

TheDJ added a comment to T197655: <mapframe> JavaScript runtime error.

wgKartographerMapServer is provided by module ext.kartographer, which is a dependency of ext.kartographer.box, but is not a dependency of ext.kartographer.settings. That seems like a mistake in the RL definitions indeed.

Mon, Jul 9, 3:26 PM · MediaWiki-ResourceLoader, Wikimedia-log-errors, Maps (Kartographer), JavaScript
TheDJ added a comment to T193455: <mapframe>: Automatic zoom and centering only works on page preview.

Another example: https://nl.wikipedia.org/w/index.php?title=Wikipedia:De_kroeg&oldid=51923141#<mapframe>
Compare the 2nd and the 3rd map in that row.

Mon, Jul 9, 3:20 PM · Maps (Kartographer)
TheDJ updated the task description for T193455: <mapframe>: Automatic zoom and centering only works on page preview.
Mon, Jul 9, 3:18 PM · Maps (Kartographer)
TheDJ added a comment to T155927: Maps that contain Wikidata ids and are stored on Commons fail to display when called via <maplink> or <mapframe>.

@Mholloway we have since concluded that T193455: <mapframe>: Automatic zoom and centering only works on page preview is a more likely match of the issue described.

Mon, Jul 9, 3:16 PM · Maps (Kartographer), Commons-Datasets
TheDJ edited Description on Discovery.
Mon, Jul 9, 1:42 PM
TheDJ edited Description on Discovery.
Mon, Jul 9, 1:41 PM
TheDJ added a comment to T197735: Kartographer full page view cuts off longer captions.

Can you please provide links for the items shown in the screenshots ?

Mon, Jul 9, 1:38 PM · Maps
TheDJ added a project to T127109: Investigate using geo coordinates in Commons photos to display: Need-volunteer.

The ground work for this has been done, but it is not actually used. Depends on volunteers building new tools that work with the production services.

Mon, Jul 9, 1:37 PM · Need-volunteer, Maps
TheDJ closed T130013: Move users of maps-related tools to production services where possible as Resolved.

Some of this has happened, esp since T172065: Hunt for Toolforge tools that load resources from third party sites became more of an issue.

Mon, Jul 9, 1:18 PM · Discovery, Cloud-Services, Maps, Toolforge
TheDJ closed T137255: Establish map search goals as Declined.

Seems this never really got off the ground

Mon, Jul 9, 1:15 PM · Maps, Discovery, Epic
TheDJ closed T137255: Establish map search goals, a subtask of T137254: Improve map search, as Declined.
Mon, Jul 9, 1:15 PM · Tracking, Maps, Epic
TheDJ closed T103801: *.tiles.wmflabs.org should be served over https as Resolved.

It seems that at some point this was resolved.

Mon, Jul 9, 1:13 PM · Cloud-Services, Maps, Discovery
TheDJ added a comment to T156330: Motorway links missing from Maps.

Not even in 8 years ?

Mon, Jul 9, 11:38 AM · Easy, Discovery, Maps
TheDJ placed T156330: Motorway links missing from Maps up for grabs.

I guess at the very least this is no longer on your plate ;)

Mon, Jul 9, 11:35 AM · Easy, Discovery, Maps
TheDJ added a comment to T156330: Motorway links missing from Maps.

That distinction is internal politics to me. Effectively, i'm waiting on a product manager to be made available to do his work.

Mon, Jul 9, 11:35 AM · Easy, Discovery, Maps
TheDJ reopened T156330: Motorway links missing from Maps as "Stalled".

It has been decided that because we don't have a dedicated user-focused product manager for maps changes we can't make material product changes for maps, so the new styles work will not be deployed.

Mon, Jul 9, 11:19 AM · Easy, Discovery, Maps
TheDJ closed T139239: tools.wp-world running sed on NFS files on tools-bastion-03 as Resolved.

Considering this closed

Mon, Jul 9, 7:36 AM · Tools

Sat, Jul 7

TheDJ added a comment to T178370: Automatic zoom and positioning doesn't work for geomasks.

Are we sure this is just with wikidata ids etc ? I see something like this with even a simple .map page

<mapframe width=300 height=300 zoom=17>
{
  "type": "ExternalData",
  "service": "page",
  "title": "Haarlem/Grote Kerk.map"
}
</mapframe>

Rendered on nl.wp, as if it is located in Algeria instead of in Utrecht, Netherlands

Sat, Jul 7, 8:19 AM · Maps (Kartographer)
TheDJ added a comment to T155927: Maps that contain Wikidata ids and are stored on Commons fail to display when called via <maplink> or <mapframe>.

Are we sure this is just with wikidata ids etc ? I see something like this with even a simple .map page

Sat, Jul 7, 8:18 AM · Maps (Kartographer), Commons-Datasets

Wed, Jul 4

TheDJ created T198781: Add toggle/state buttons to WikiEditor.
Wed, Jul 4, 8:02 AM · Accessibility, CodeEditor, MediaWiki-extensions-CodeMirror, WikiEditor, Community-Tech
TheDJ added a comment to T197534: CodeMirror toggle buttons need to indicate the current state.

asked for a distinguished tooltip which tells a blind user the current state of the toggle.

We don't use tooltips for that, we use aria state, the screenreader should be able to communicate this information just fine.

Wed, Jul 4, 7:59 AM · User-Ryasmeen, VisualEditor, VisualEditor-MediaWiki-2017WikitextEditor, WikiEditor, MediaWiki-extensions-CodeMirror, Accessibility
TheDJ awarded T148030: OOUI indexed dialogs (IndexLayout) / tabbed navigation (TabPanelLayout) is not accessible a Love token.
Wed, Jul 4, 7:39 AM · OOUI (OOUI-0.27.5), Accessibility

Fri, Jun 29

TheDJ added a project to T198476: Some edits are not automatically marked as patrolled: Regression.
Fri, Jun 29, 2:47 PM · Regression, Commons

Wed, Jun 27

TheDJ added a comment to T197065: New Filters for Edit Review: UI displays poorly when the window is narrow.

Some minor bootstrap like changes:

        /* equal cells which wrap, with 1em of space in between */
	.mw-rcfilters-ui-row {
		display:flex;
		flex-wrap: wrap;
		justify-content: space-between;
		margin: 0 -0.5em;
	}
	.mw-rcfilters-ui-cell {
		flex: 1 1;
		padding: 0 0.5em;
	}
        /* overrides */
	.mw-rcfilters-ui-watchlistTopSectionWidget-watchlistDetails {
		min-width: 20em;
	}
	.mw-rcfilters-ui-watchlistTopSectionWidget-editWatchlistButton,
	.mw-rcfilters-ui-filterTagMultiselectWidget-cell-save,
	.mw-rcfilters-ui-filterTagMultiselectWidget-cell-reset,
	.mw-rcfilters-ui-filterTagMultiselectWidget-views-select,
	.mw-rcfilters-ui-watchlistTopSectionWidget-savedLinks,
	.mw-rcfilters-ui-itemMenuOptionWidget-highlightButton,
	.mw-rcfilters-ui-itemMenuOptionWidget-excludeLabel,
	.mw-rcfilters-ui-rcTopSectionWidget-savedLinks{
		align-self: flex-end;
		flex: 0 1;
	}
	.mw-rcfilters-ui-filterTagMultiselectWidget-views-select {
		padding-left: 0;
	}
	.mw-rcfilters-ui-filterTagMultiselectWidget-views-input {
		padding-right: 0;
	}
	.mw-rcfilters-ui-filterTagMultiselectWidget-views-select {
		/* This a bug in the current styling ? */
		width: auto;
	}
	.mw-rcfilters-ui-watchlistTopSectionWidget-editWatchlistButton .oo-ui-buttonWidget {
		margin-left: 0;
	}


same style, without even using media queries to further optimize

Wed, Jun 27, 7:58 AM · Collaboration-Team-Triage, MonoBook, Timeless, Edit-Review-Improvements-Integrated-Filters
TheDJ added a comment to T198276: VE Wikitext editor cursor is misplaced on Minerva desktop.

The surfaces ve-ce-documentNode and CodeMirror don't overlap. I could fix the problem by setting a 1.5em padding on CodeMirror, to match what the document node has in padding. We need to check how the settings are adapted for things like monobook.

Wed, Jun 27, 7:50 AM · MediaWiki-extensions-CodeMirror, Community-Tech, VisualEditor, MinervaNeue (Desktop)

Sun, Jun 24

TheDJ added a comment to T195861: Create a committee to improve the math support in Wikimedia projects.

@Deneben No they are hard requirements for all new software deployed to Wikimedia properties, and we are continuously fixing and reworking the last of the existing code that doesn't comply with it yet. JS causing reflowing is the latest focus of the now 6 year effort to improve performance of MediaWiki and Wikipedia.

Sun, Jun 24, 11:10 AM · User-mobrovac, Patch-For-Review, Math

Sat, Jun 23

TheDJ added a comment to T198010: jquery.textSelection methods work incorrectly with 2017 wikitext editor.

Please note your browser and version, the wiki in question etc etc etc.

Sat, Jun 23, 2:19 PM · VisualEditor (Current work), Patch-For-Review, VisualEditor-MediaWiki-2017WikitextEditor
TheDJ added a comment to T178990: Support new Filters for Recent Changes release on wikis that have FlaggedRevisions.

Dear TheDJ,

At English Wikinews we are being told a configuration change would be made by WMF even though local consensus (so far I think about 4 people had their say and said 'no') is against it. Surely this should not happen? I was assuming there is an open ticket somewhere and this is it.

Sat, Jun 23, 2:17 PM · User-notice-collaboration, User-notice, Community-Relations-Support (Oct-Dec 2017), Edit-Review-Improvements-Integrated-Filters, Collaboration-Team-Triage
TheDJ updated subscribers of T20871: Include at least some EXIF metadata in resized pictures.

All filetypes ?

Sat, Jun 23, 8:56 AM · WMF-Legal, Commons, Multimedia, MediaWiki-File-management
TheDJ added a comment to T178990: Support new Filters for Recent Changes release on wikis that have FlaggedRevisions.

I think we are having difficulty communicating about this task at English Wikinews (on the list at P6067).

Sat, Jun 23, 8:51 AM · User-notice-collaboration, User-notice, Community-Relations-Support (Oct-Dec 2017), Edit-Review-Improvements-Integrated-Filters, Collaboration-Team-Triage
TheDJ added a comment to T197065: New Filters for Edit Review: UI displays poorly when the window is narrow.

It's actually bothering me, because i use vector in responsive mode, so i might get around to it at some point ;)

Sat, Jun 23, 8:48 AM · Collaboration-Team-Triage, MonoBook, Timeless, Edit-Review-Improvements-Integrated-Filters
TheDJ added a comment to T197065: New Filters for Edit Review: UI displays poorly when the window is narrow.

This feels like something that should be fixed with some flexbox magic. ;)

Sat, Jun 23, 8:46 AM · Collaboration-Team-Triage, MonoBook, Timeless, Edit-Review-Improvements-Integrated-Filters
TheDJ added a comment to T195861: Create a committee to improve the math support in Wikimedia projects.

Then we force everyone who should be concerned about it to give us feedback before putting further work into it.

@TheDJ Since you were involved in the client-side mathjax project of the past, it would be nice if you could help us to avoid problems that killed it last time.

Sat, Jun 23, 7:45 AM · User-mobrovac, Patch-For-Review, Math

Tue, Jun 19

TheDJ added a comment to T197555: Change syntaxhighlight icon to something else than a previously signature pensil.

@Base please don't use words as 'corrupted'. It's not conductive to the conversation.

Tue, Jun 19, 9:26 AM · MediaWiki-extensions-CodeMirror, Design
TheDJ added a comment to T197622: [wmf.8] Misplaced 'x' on the first load on Watchlist page .

FYI, production has a similar problem in Monobook.

Tue, Jun 19, 9:21 AM · User-notice, OOUI, Edit-Review-Improvements-Integrated-Filters, Collaboration-Team-Triage
TheDJ added a comment to T197534: CodeMirror toggle buttons need to indicate the current state.

@matmarex the state is, it works, it's accessible, but it's just... a kludge on top of wiki editor ;)

Tue, Jun 19, 8:30 AM · User-Ryasmeen, VisualEditor, VisualEditor-MediaWiki-2017WikitextEditor, WikiEditor, MediaWiki-extensions-CodeMirror, Accessibility

Mon, Jun 18

TheDJ added a comment to T174635: Middle-click to paste text doesn't work in CodeMirror on Linux.

@Pamputt I consider https://github.com/codemirror/CodeMirror/issues/931 to be the upstream ticket and it clearly indicates that it is not yet fixed, and that the FF fixes are imcomplete.

Mon, Jun 18, 8:36 AM · Upstream, MediaWiki-extensions-CodeMirror

Sun, Jun 17

TheDJ closed T13627: Add syntax highlighting to Edit box as Resolved.

With MediaWiki-extensions-CodeMirror now deployed, I'm calling this fixed.

Sun, Jun 17, 9:23 PM · MediaWiki-Page-editing
TheDJ added a comment to T134269: Allow editors to add and see existing soft hypen (&shy;) non-visible characters where needed.

Relates to T181677: Implement syntax highlight for U+00A0 (no-break space, nbsp)

Sun, Jun 17, 9:18 PM · Design, VisualEditor-EditingTools, VisualEditor-ContentLanguage, VisualEditor
TheDJ added a parent task for T170004: Syntax Highlighting: Translated magic words aren't recognized in Hebrew: T170001: Support CodeMirror syntax highlighting on RTL wikis.
Sun, Jun 17, 9:16 PM · MediaWiki-extensions-CodeMirror
TheDJ added a subtask for T170001: Support CodeMirror syntax highlighting on RTL wikis: T170004: Syntax Highlighting: Translated magic words aren't recognized in Hebrew.
Sun, Jun 17, 9:16 PM · I18n, RTL, MediaWiki-extensions-CodeMirror
TheDJ added a subtask for T166098: Syntax sensitive and highlighting source code editor for template code wanted: T30684: Collapse templates in editor.
Sun, Jun 17, 9:12 PM · MediaWiki-extensions-CodeMirror, MediaWiki-Page-editing, MediaWiki-Templates
TheDJ added a parent task for T30684: Collapse templates in editor: T166098: Syntax sensitive and highlighting source code editor for template code wanted.
Sun, Jun 17, 9:12 PM · Community-Tech, MediaWiki-extensions-CodeMirror, MediaWiki-Page-editing
TheDJ added a comment to T181677: Implement syntax highlight for U+00A0 (no-break space, nbsp).

https://github.com/coderaiser/cm-show-invisibles

Sun, Jun 17, 9:05 PM · Patch-For-Review, MediaWiki-extensions-CodeMirror
TheDJ closed T175961: CodeMirror on NWE: pasting problems from one surface to another - line size? as Resolved.

i'm pretty sure this was fixe a while ago.

Sun, Jun 17, 9:03 PM · User-Ryasmeen, VisualEditor-MediaWiki-2017WikitextEditor, MediaWiki-extensions-CodeMirror, VisualEditor
TheDJ added a comment to T192019: CodeMirror syntax highlighting shows overlapping text when word-wrap (hyphens, word-break) is overridden.

So is this fixed now with T192019#4134117 ?

Sun, Jun 17, 8:57 PM · MW-1.31-release-notes (WMF-deploy-2018-04-17 (1.31.0-wmf.30)), Patch-For-Review, Easy, Community-Tech, VisualEditor, VisualEditor-MediaWiki-2017WikitextEditor, MediaWiki-extensions-CodeMirror
TheDJ added a project to T197239: CodeMirror – icon extends from bar: WikiEditor.

I've checked. The other buttons have the same issue, it's just that they don't have an outline and backgroundcolor set as this button does in the active state.

Sun, Jun 17, 8:55 PM · WikiEditor, MediaWiki-extensions-CodeMirror, Community-Tech
TheDJ triaged T196512: CodeMirror load breaks at $button.toggleClass(...) as Low priority.
Sun, Jun 17, 8:37 PM · Community-Tech, MediaWiki-extensions-CodeMirror
TheDJ added a comment to T196512: CodeMirror load breaks at $button.toggleClass(...) .

Ah, I found this too.

Sun, Jun 17, 8:37 PM · Community-Tech, MediaWiki-extensions-CodeMirror
TheDJ moved T174635: Middle-click to paste text doesn't work in CodeMirror on Linux from Backlog to Reported Upstream on the Upstream board.
Sun, Jun 17, 8:25 PM · Upstream, MediaWiki-extensions-CodeMirror
TheDJ changed the status of T174635: Middle-click to paste text doesn't work in CodeMirror on Linux from Open to Stalled.

This ticket is stalled on the upstream issue.

Sun, Jun 17, 8:25 PM · Upstream, MediaWiki-extensions-CodeMirror
TheDJ triaged T197534: CodeMirror toggle buttons need to indicate the current state as Normal priority.
Sun, Jun 17, 8:23 PM · User-Ryasmeen, VisualEditor, VisualEditor-MediaWiki-2017WikitextEditor, WikiEditor, MediaWiki-extensions-CodeMirror, Accessibility
TheDJ added a comment to T197534: CodeMirror toggle buttons need to indicate the current state.

Hmm, this proved more annoying than i figured. Notes

Sun, Jun 17, 8:14 PM · User-Ryasmeen, VisualEditor, VisualEditor-MediaWiki-2017WikitextEditor, WikiEditor, MediaWiki-extensions-CodeMirror, Accessibility
TheDJ added a comment to T197534: CodeMirror toggle buttons need to indicate the current state.

Just tried it again in FF on my mac, and for me tab moves to the editsummary. This matches what is defined in the code, which basically tells CodeMirror to ignore interpreting tabs (Same spot as where my patch will disable shift-tab)

Sun, Jun 17, 7:41 PM · User-Ryasmeen, VisualEditor, VisualEditor-MediaWiki-2017WikitextEditor, WikiEditor, MediaWiki-extensions-CodeMirror, Accessibility
TheDJ triaged T197263: insertTags shift cursor position to start of the line as Low priority.
Sun, Jun 17, 4:12 PM · Patch-For-Review, Community-Tech, MediaWiki-extensions-CodeMirror
TheDJ added a comment to T197534: CodeMirror toggle buttons need to indicate the current state.

With CodeMirror enabled, ⇥ in edit textarea has another effect and does not navigate any longer through the form

Sun, Jun 17, 3:47 PM · User-Ryasmeen, VisualEditor, VisualEditor-MediaWiki-2017WikitextEditor, WikiEditor, MediaWiki-extensions-CodeMirror, Accessibility

Jun 15 2018

TheDJ added a comment to T197087: Remove or limit edituserjs and similar rights.

There is also the ease of it--TheDJ would have to fork the page in question, make the changes, then ask the user to copy his changes, and then nuke whatever copy he had made, if he could not edit the pages directly. This ease is valuable (for the same reason that it is a dangerous vector).

Jun 15 2018, 9:26 PM · MediaWiki-Interface, Security-Core
TheDJ created T197463: Centralnotices leaves lots of entries in LocalStorage.
Jun 15 2018, 11:31 AM · MediaWiki-extensions-CentralNotice
TheDJ added a comment to T189806: Deploy GlobalPrefs on production.

@Niharika now filed separately as T197450: test.wp is using test2.wp's message cache

Jun 15 2018, 9:26 AM · Patch-For-Review, Community-Tech, Wikimedia-Site-requests, MediaWiki-extensions-GlobalPreferences
TheDJ created T197450: test.wp is using test2.wp's message cache.
Jun 15 2018, 9:26 AM · Performance-Team, Gadgets, MediaWiki-Cache, Operations

Jun 14 2018

TheDJ added a project to T30684: Collapse templates in editor: MediaWiki-extensions-CodeMirror.

This should probably be moved to the CodeMirror functionality now.

Jun 14 2018, 8:48 PM · Community-Tech, MediaWiki-extensions-CodeMirror, MediaWiki-Page-editing
TheDJ added a comment to T197263: insertTags shift cursor position to start of the line.

It's not the same issue I think. I had breakpoints on all the textSelection functions and this setSelection is the last in the sequence, and I just verified with codemirror.doc.getCursor() in the console that the setFocus() causes the move of the cursor.

Jun 14 2018, 8:13 PM · Patch-For-Review, Community-Tech, MediaWiki-extensions-CodeMirror
TheDJ added a comment to T184857: Problems with editing huge pages.

workTime, workDelay: number
Highlighting is done by a pseudo background-thread that will work for workTime milliseconds, and then use timeout to sleep for workDelay milliseconds. The defaults are 200 and 300, you can change these options to make the highlighting more or less aggressive.

Jun 14 2018, 8:05 PM · MediaWiki-extensions-CodeMirror, VisualEditor-Performance, VisualEditor, VisualEditor-MediaWiki-2017WikitextEditor
TheDJ created T197263: insertTags shift cursor position to start of the line.
Jun 14 2018, 7:58 PM · Patch-For-Review, Community-Tech, MediaWiki-extensions-CodeMirror
TheDJ awarded T197070: +2 for MarcoAurelio in mediawiki/* a Like token.
Jun 14 2018, 7:42 AM · Repository-Ownership-Requests
TheDJ added a comment to T189806: Deploy GlobalPrefs on production.

Not sure where this project is, but I just noticed that on test.wikipedia.org, I see the gadgets from test2.wikipedia.org

Jun 14 2018, 7:40 AM · Patch-For-Review, Community-Tech, Wikimedia-Site-requests, MediaWiki-extensions-GlobalPreferences

Jun 13 2018

TheDJ added a comment to T197035: Do not print navboxes.

Ideally, it would be cool if we could detect problems like this, so that local communities can be more aware. Something like Special:Linter but for common mobile and print problems that we detect and then linking to advise on how to fix etc..

Jun 13 2018, 8:45 AM · User-Jdlrobson, Readers-Web-Backlog (Tracking), Reading-Web-Local-Wiki-Issues, Proton, Vector
TheDJ added a comment to T196721: Make non-responsive Monobook the default for German-language Wikipedia.

wasn't page previews set like that because it'd conflict

No, it has a separate mechanism for that, which just stops it from presenting when it detects that Navigation Popups is active (so not really disabled, just not showing).

Jun 13 2018, 8:40 AM · MonoBook

Jun 12 2018

TheDJ added a comment to T197035: Do not print navboxes.

enwiki hides them through https://en.wikipedia.org/wiki/MediaWiki:Print.css

Jun 12 2018, 11:11 PM · User-Jdlrobson, Readers-Web-Backlog (Tracking), Reading-Web-Local-Wiki-Issues, Proton, Vector
TheDJ added a comment to T196893: Restore Watchlist functionality to grade C browsers e.g. Windows Phone 8.

Why doesnt the watchstar work without javascript ? No link fallback ? Shouldnt it have that ?

Jun 12 2018, 7:14 PM · Readers-Web-Backlog, MobileFrontend

Jun 11 2018

Gerrit Code Review <gerrit@wikimedia.org> committed rEWIS68b66cac5e9b: Update patch set 1 (authored by TheDJ).
Update patch set 1
Jun 11 2018, 6:28 AM
Gerrit Code Review <gerrit@wikimedia.org> committed rEWIS22bade89e4c2: Update patch set 1 (authored by TheDJ).
Update patch set 1
Jun 11 2018, 6:28 AM

Jun 10 2018

Gerrit Code Review <gerrit@wikimedia.org> committed rESCC9d54312dd5a6: Update patch set 1 (authored by TheDJ).
Update patch set 1
Jun 10 2018, 7:27 AM
Gerrit Code Review <gerrit@wikimedia.org> committed rERSLbf526ef360f3: Update patch set 1 (authored by TheDJ).
Update patch set 1
Jun 10 2018, 3:31 AM

Jun 8 2018

Gerrit Code Review <gerrit@wikimedia.org> committed rQUIBBLE77c1eb7ff672: Update patch set 1 (authored by TheDJ).
Update patch set 1
Jun 8 2018, 6:20 PM

Jun 7 2018

TheDJ added a comment to T195256: Implement accessibility jump links with pure CSS.

@Volker_E the intent was using it for ourselves, not for script authors. Easier to find back than a freeform todo comment in some random repo. Also scannable with automation tools within the live website, which seems like it could be useful

Jun 7 2018, 4:28 PM · MW-1.32-release-notes (WMF-deploy-2018-06-12 (1.32.0-wmf.8)), Performance-Team (Radar), Patch-For-Review, Accessibility
TheDJ added a comment to T195256: Implement accessibility jump links with pure CSS.

@Krinkle As i was reading your commit i came up with #jump-to-nav:not(:empty)
I was also thinking: If we leave dom in place that is deprecated, for the sake of scripts... What if we add a class="mw-deprecated" or something to that ?

Jun 7 2018, 2:02 PM · MW-1.32-release-notes (WMF-deploy-2018-06-12 (1.32.0-wmf.8)), Performance-Team (Radar), Patch-For-Review, Accessibility

Jun 5 2018

TheDJ added a comment to T196215: Responsive Layout for Monobook broken when standard user scripts are used.

I'm not very familiar with the way MediaWiki is structured, so I'm curious to learn more. Why does mw core care about the CSS of a skin?

Jun 5 2018, 10:07 AM · User-notice, MonoBook

Jun 4 2018

TheDJ added a comment to T195256: Implement accessibility jump links with pure CSS.

Hmm, I think we need to remove:

	#jump-to-nav {
		/* Negate #contentSub's margin and replicate it so that the jump to links don't affect the spacing */
		margin-top: -1.4em;
		margin-bottom: 1.4em;
	}

https://github.com/wikimedia/mediawiki/blob/master/resources/src/mediawiki.skinning/interface.css#L66

Jun 4 2018, 8:14 PM · MW-1.32-release-notes (WMF-deploy-2018-06-12 (1.32.0-wmf.8)), Performance-Team (Radar), Patch-For-Review, Accessibility

Jun 2 2018

TheDJ added a comment to T196213: Set mobile breakpoint smaller for responsive MonoBook.

You can also consider to query on device-width (which should be zoom level proof right?). It's generally bad practice to do that, but here it might be worthwile. or combine both.

Jun 2 2018, 8:26 AM · Patch-For-Review, MonoBook

Jun 1 2018

TheDJ added a comment to T194814: Reduce amount of headers sent from web responses.
X-Varnish	521726689 533337780, 225083667 220092282, 525815818 515121340
Server	mw1238.eqiad.wmnet

Especially these two can be handy. In the past I've used them to quickly help identify everything from varnish servers that got cut of from the cache invalidation streams, to accidentally pooled servers which were running outdated or incorrectly configured setups.

Jun 1 2018, 6:19 AM · Performance-Team (Radar), Patch-For-Review, media-storage, Operations, Traffic
TheDJ updated subscribers of T184871: Add encoding HTML header to LilyPond output.

Fixing cors or just adding the right mime type, probably requires some work with the media-storage ppl / someone with some swift file backend experience.

Jun 1 2018, 5:53 AM · I18n, MW-1.32-release-notes (WMF-deploy-2018-05-29 (1.32.0-wmf.6)), Patch-For-Review, MediaWiki-extensions-Score
TheDJ updated the task description for T66819: PDF of article does not display "[who?]" as superscript but "Wikipedia:Avoid weasel words" instead.
Jun 1 2018, 5:47 AM · Proton, Collection
TheDJ added a comment to T184871: Add encoding HTML header to LilyPond output.

Might be a CORS issue, as its a crossdomain link...

Jun 1 2018, 4:14 AM · I18n, MW-1.32-release-notes (WMF-deploy-2018-05-29 (1.32.0-wmf.6)), Patch-For-Review, MediaWiki-extensions-Score

May 31 2018

TheDJ placed T196043: wikidiff2 repo doesn't have LICENSE in source nor distribution, nor individual files up for grabs.
May 31 2018, 7:54 AM · Patch-For-Review, Software-Licensing, wikidiff2