Page MenuHomePhabricator

IKhitron (Igal)
User

Projects

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Sunday

  • Clear sailing ahead.

User Details

User Since
Jan 4 2015, 2:23 PM (268 w, 4 d)
Availability
Available
LDAP User
IKhitron
MediaWiki User
IKhitron [ Global Accounts ]

Recent Activity

Fri, Feb 21

IKhitron added a comment to T245298: 107: Make empty lists list collapsible and collapsed.

{{done}}

Fri, Feb 21, 9:54 AM · User-DannyS712, DannyS712-Global_watchlist.js

Thu, Feb 20

IKhitron added a comment to T245428: 113: The script can't return from empty lists.

Continuing working. Opened two windows on hewiki with GL on the same wiki, emptied all, waited to a new edit. Clicked Refresh on the first, get nothing. Clicked F5 on the second, get the result. After running a query on both consules, get the same output:

var api = new mw.ForeignApi( 'https://ru.wikipedia.org/w/api.php' );
api.get( {
    action: 'query',
    list: 'watchlist',
    wlshow: 'unread'
} ).done( function ( data ) {
    console.log( data );
} );
Object { abort: abort(), state: state(), always: always(), catch: catch(), pipe: pipe(), then: then(), promise: promise(), progress: add(), done: add(), fail: add()
 }
Thu, Feb 20, 3:12 PM · User-DannyS712, DannyS712-Global_watchlist.js

Tue, Feb 18

IKhitron added a comment to T245428: 113: The script can't return from empty lists.

Well, working on this in meta:User:IKhitron/Global Watchlist.js. Now I'm certain that the problem is in script, in api usage, and not in api itself. This what I have now on a console, while there is no wikidata in the list:


and this what I have in another tab, after F5:

As you can see, there is the same wikidata item in both, Q9095390. So, the script is aware about this edit, it just do not show it. So, the task closing was too soon. Continuing to work on this.

Tue, Feb 18, 12:56 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245428: 113: The script can't return from empty lists.

Yes, you can. You can remove the Refresh button, because it can't work as promised. In any case, I'll try to find some time to investigate this. Seems quite impossible that api returns different answers in different tabs at the same time, depends on the screen layout, which this api even does not need to see. Maybe the api just gets wrong parameters from the script in this case - timestamp or something else and as a result of this return wrong answer. Especially when we can see the right answer on the console, and it must come from somewhere, it is new, so couldn't remain from before. @Masumrezarock100, can you help in this investigation, please?

Tue, Feb 18, 10:33 AM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245428: 113: The script can't return from empty lists.

Nevermind why. The result is that the button does not do what it supposed to do.

Tue, Feb 18, 2:04 AM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245428: 113: The script can't return from empty lists.

So it will remain there, but will not work?

Tue, Feb 18, 12:35 AM · User-DannyS712, DannyS712-Global_watchlist.js

Mon, Feb 17

IKhitron added a comment to T245428: 113: The script can't return from empty lists.

Exactly what I said, you can't use the refresh button, because api does not return the right answer.

Mon, Feb 17, 11:51 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245428: 113: The script can't return from empty lists.

I do not understand. Do you mean that you're going to remove the refresh button entirely, because api does not support its functionality? And there us nothing you can do?

Mon, Feb 17, 11:46 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245312: 109: Make setTimeout value for live updates configurable.

I understand now. Thanks, I'll think if I can do it.

Mon, Feb 17, 11:44 PM · patch-welcome, User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245455: 114: Open compound diff in new tab.

Funny. Of course, but all I want is avoid doing this hundreds of times every day. Now I run attr in console every time. Unfortunately, I couldn't find a way to catch the moment when all the lists are shown, so javascript could do something.

Mon, Feb 17, 10:30 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245298: 107: Make empty lists list collapsible and collapsed.

Sorry. I meant @DannyS712.

Mon, Feb 17, 7:39 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245428: 113: The script can't return from empty lists.

One more observation. I did a lot of tests in the last couple of hours, and can focus the problem much better.

  1. Open the GW.
  2. Open in other tabs and read all the shown diffs for some wiki.
  3. Click refresh.
  4. See the list is gone to globalWatch-col.
  5. Wait until there will be at least one new edit from that wiki.
  6. Click refresh.
  7. Expected: The list moves from globalWatch-col up and shows the new edits.
  8. Got: The list remains in globalWatch-col without showing edits.
Mon, Feb 17, 7:38 PM · User-DannyS712, DannyS712-Global_watchlist.js
Masumrezarock100 awarded T245455: 114: Open compound diff in new tab a Like token.
Mon, Feb 17, 7:18 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron changed the subtype of T245455: 114: Open compound diff in new tab from "Task" to "Feature Request".
Mon, Feb 17, 7:07 PM · User-DannyS712, DannyS712-Global_watchlist.js
Restricted Application assigned T245455: 114: Open compound diff in new tab to DannyS712.
Mon, Feb 17, 7:07 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245428: 113: The script can't return from empty lists.

Thanks a lot.
P10428

Mon, Feb 17, 7:03 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron created P10428 T245428 console.
Mon, Feb 17, 7:03 PM · DannyS712-Global_watchlist.js
IKhitron added a comment to T245312: 109: Make setTimeout value for live updates configurable.

OK, but once again, if the user can set the new value in global.js only, why do you need to do something with settings at all? If there is a new value in global.js, it will be used on each browser refresh, otherwise it will not. Why do you need to keep it in user settings if you'll never use this setting?

Mon, Feb 17, 6:01 PM · patch-welcome, User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245428: 113: The script can't return from empty lists.

For example,

[email protected]:getWikidataLabels wdlabels: index.php:195:11
{…}
​
Q17523682: Object { type: "item", id: "Q17523682", labels: {…} }
​
<prototype>: Object { … }
index.php:196:11

About the debugLog, where do you want me to paste it?

Mon, Feb 17, 5:58 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245312: 109: Make setTimeout value for live updates configurable.

There is no timeout field on the settings page, but you can still manually set one

Mon, Feb 17, 5:54 PM · patch-welcome, User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245312: 109: Make setTimeout value for live updates configurable.
  • If the user has manually set a timeout by previously editing the settings, the timeout should be kept when saving again

The user cannot set a timeout by editing the settings, because there is no timeout field in the settings.

Mon, Feb 17, 5:53 PM · patch-welcome, User-DannyS712, DannyS712-Global_watchlist.js
IKhitron updated subscribers of T245298: 107: Make empty lists list collapsible and collapsed.

@Danny, we're talking about different things here. Let me try to explain once again. I do not talk about pinning. I talk about the state when all edits lists will start un-collapsed and only the bottom list will start collapsed, because no one needs it, except for special occasions.

Mon, Feb 17, 5:51 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245428: 113: The script can't return from empty lists.

Do you mean to spend an hour to open all the objects again, or the new one you've added?

Mon, Feb 17, 5:49 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245312: 109: Make setTimeout value for live updates configurable.

No apologies are needed. I'm the stupid one here.

Mon, Feb 17, 1:47 PM · patch-welcome, User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245312: 109: Make setTimeout value for live updates configurable.

I just quoted the last line. Not meant that my answer is to this line only.

Mon, Feb 17, 1:39 PM · patch-welcome, User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245312: 109: Make setTimeout value for live updates configurable.

The meaning of this part should be obvious.

Mon, Feb 17, 1:19 PM · patch-welcome, User-DannyS712, DannyS712-Global_watchlist.js
Restricted Application assigned T245428: 113: The script can't return from empty lists to DannyS712.
Mon, Feb 17, 1:02 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245312: 109: Make setTimeout value for live updates configurable.

It'll be awhile before I'm motivated to do this, because I'm not planning to use it myself, but if anyone wants to submit a patch please do so:

  • Can be submitted as either a git patch/diff, or a fork on enwiki or meta (so that I can use Special:ComparePages)
  • The config setting should not be added to a user's preferences if you are using the default
  • The config setting should be kept when saving new settings
  • No new field should be added to the settings page
Mon, Feb 17, 12:32 PM · patch-welcome, User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245298: 107: Make empty lists list collapsible and collapsed.

So, what do you say, @DannyS712?

Mon, Feb 17, 10:40 AM · User-DannyS712, DannyS712-Global_watchlist.js

Sat, Feb 15

IKhitron added a comment to T245298: 107: Make empty lists list collapsible and collapsed.

Sure, but this is much simpiler that T245348.

Sat, Feb 15, 6:22 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245298: 107: Make empty lists list collapsible and collapsed.

Exactly. This was the purpose of this task. This


is too much.

Sat, Feb 15, 5:50 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron updated the task description for T245348: 112: The state of collapsible elements changes to expand after clicking refresh.
Sat, Feb 15, 5:36 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245298: 107: Make empty lists list collapsible and collapsed.

Not at all. It can be good, but it is not a part of this task. All I'm saying that the script should have something like

$('.globalWatch-col').addClass('mw-collapsed');
Sat, Feb 15, 5:35 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245298: 107: Make empty lists list collapsible and collapsed.

That's not an unexpected behavior since GW doesn't track whether people have collapsed the list or not. I'd call this task resolved since the criteria for sign-off has been met. Feel free to open a new task if you'd like a pinning function. I think cookies can be used to track whether a user have collapsed the list. See an example, RevisionSlider extension.
The styling of the collapse link could be made similar to it.


The screenshot of https://meta.wikimedia.org/w/index.php?diff=19810190&oldid=19808405&diffmode=source.

Sat, Feb 15, 4:38 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron reopened T245298: 107: Make empty lists list collapsible and collapsed, a subtask of T245336: 111: Newsletter, issue 6, as Open.
Sat, Feb 15, 2:01 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron reopened T245298: 107: Make empty lists list collapsible and collapsed as "Open".
Sat, Feb 15, 2:01 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245326: 110: Store user preferences as a global preference.

@IKhitron Danny haven't had enough time to work on it. So feel free to claim this task if you want to work on it. 😀 There indeed seems to be a need for deprecating the current way of saving GW preferences.

Sat, Feb 15, 1:43 AM · User-DannyS712, DannyS712-Global_watchlist.js

Fri, Feb 14

IKhitron added a comment to T245312: 109: Make setTimeout value for live updates configurable.

But Masumrezarock100's idea about a mnimum is very good. If somebody will set it to 5 milliseconds it will blow up the script.

Fri, Feb 14, 11:52 PM · patch-welcome, User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245312: 109: Make setTimeout value for live updates configurable.

Fair enough.

Fri, Feb 14, 11:50 PM · patch-welcome, User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245312: 109: Make setTimeout value for live updates configurable.

Again, I do not search a solution for global watchlist or for rcfilters. It happens with every heavy script. Just a way to configure the number. I already made a local version of global watchlist settings to avoid usage of global.js. I really do not want to have a local version of global watchlist itself just for this number.

Fri, Feb 14, 11:43 PM · patch-welcome, User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245312: 109: Make setTimeout value for live updates configurable.

No error. Just takes time and resources. I do not ask to fix the script. I understand it needs all this, because of many api calls. So all I ask is an opportunity to change the number.

Fri, Feb 14, 11:17 PM · patch-welcome, User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245312: 109: Make setTimeout value for live updates configurable.

And about the rc-filters - the mobile need reboot after a couple of minutes.

Fri, Feb 14, 10:56 PM · patch-welcome, User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245312: 109: Make setTimeout value for live updates configurable.

Exactly. Not all the users have money for fast connection or for enough data usage limit. I have 4G on a good Galaxy, and clicking on Refresh makes the global watchlist disappear for 3 to 25 seconds. And I have only 10Gb montly, so I can't spent it for api calls every 7.5 seconds, when I do not need this data more than once in 10 minutes.

Fri, Feb 14, 10:55 PM · patch-welcome, User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245312: 109: Make setTimeout value for live updates configurable.

@IKhitron are you having any problems with the current setTimout value? I am curious to know why you need to jump to 600000ms.

Fri, Feb 14, 10:38 PM · patch-welcome, User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245312: 109: Make setTimeout value for live updates configurable.

Do not care how. And there can be a hardcoded minimum of 7500. But one can't use live updates on mobile with this value, exactly from the same reason I can't use rcfilters.

Fri, Feb 14, 10:37 PM · patch-welcome, User-DannyS712, DannyS712-Global_watchlist.js
IKhitron renamed T245312: 109: Make setTimeout value for live updates configurable from Make setTimeout value for live updatesv configurable to Make setTimeout value for live updates configurable.
Fri, Feb 14, 10:07 PM · patch-welcome, User-DannyS712, DannyS712-Global_watchlist.js
IKhitron changed the subtype of T245312: 109: Make setTimeout value for live updates configurable from "Task" to "Feature Request".
Fri, Feb 14, 10:00 PM · patch-welcome, User-DannyS712, DannyS712-Global_watchlist.js
Restricted Application assigned T245312: 109: Make setTimeout value for live updates configurable to DannyS712.
Fri, Feb 14, 10:00 PM · patch-welcome, User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245300: 108: Move the settings from global.js to personal json page..

Marking as declined in favor of T238299 per comments in T237082#5680479. I agree with making the settings not public. Setting up a global preference would be better.

Fri, Feb 14, 9:45 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron changed the subtype of T245300: 108: Move the settings from global.js to personal json page. from "Task" to "Feature Request".
Fri, Feb 14, 8:17 PM · User-DannyS712, DannyS712-Global_watchlist.js
Restricted Application assigned T245300: 108: Move the settings from global.js to personal json page. to DannyS712.
Fri, Feb 14, 8:16 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245298: 107: Make empty lists list collapsible and collapsed.

Yes. For me, it will look much better. I have 17 sites like these at this moment, for example. And it should be very easy.

Fri, Feb 14, 8:11 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron changed the subtype of T245298: 107: Make empty lists list collapsible and collapsed from "Task" to "Feature Request".
Fri, Feb 14, 8:07 PM · User-DannyS712, DannyS712-Global_watchlist.js
Restricted Application assigned T245298: 107: Make empty lists list collapsible and collapsed to DannyS712.
Fri, Feb 14, 8:07 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245104: 106: Strikethrough fails on links.

Much better, thanks!

Fri, Feb 14, 12:21 PM · Regression, DannyS712-Global_watchlist.js, User-DannyS712

Thu, Feb 13

IKhitron added a comment to T245104: 106: Strikethrough fails on links.

I think I found. It should be white-space: nowrap; instead. Looks like it does resolve the problem.
So, the full answer is replacing

.globalWatch-site a {
	unicode-bidi: isolate;
	display: inline-block;
}

to

.globalWatch-site a {
	unicode-bidi: isolate;
	display: inline;
        white-space: nowrap;
}

if there is a need in display: inline; as explained above, or

.globalWatch-site a {
	unicode-bidi: isolate;
        white-space: nowrap;
}

otherwise.

Thu, Feb 13, 1:14 PM · Regression, DannyS712-Global_watchlist.js, User-DannyS712
IKhitron added a comment to T245104: 106: Strikethrough fails on links.

Well, as a first step you can replace inline-block to be inline, or remove the rule entirely, if it was inline before the change. It will fix the strikethrough problem, but return a small rtl problem. I'll continue to seek a permanent solution.

Thu, Feb 13, 1:01 PM · Regression, DannyS712-Global_watchlist.js, User-DannyS712
IKhitron added a comment to T245104: 106: Strikethrough fails on links.

Suspect changes in CSS as part of T245042: 103: Make the script RTL-compatible may be the issue
@IKhitron any idea what may have caused this?

Thu, Feb 13, 10:14 AM · Regression, DannyS712-Global_watchlist.js, User-DannyS712
IKhitron added a comment to T245051: 104: Add the invocation link to all skins.

Very well thou.

Thu, Feb 13, 10:09 AM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245051: 104: Add the invocation link to all skins.

Actually, maybe you'd like to check the timeless skin by yourself and decide if it's a good idea to use p-views there, even it's possible, because the link appears in very surprising spot.

Thu, Feb 13, 1:58 AM · User-DannyS712, DannyS712-Global_watchlist.js

Wed, Feb 12

IKhitron added a comment to T243489: 97: Port GlobalWatchlist.js into a gadget [spike].

I wonder what exact problem you were facing.

The console shows a message "unclosed parentheses". Tried several variants, the message is the same, the location of opening "(" is different, but accordand ")" is always very far in the code. So my first thought was that not all the lines are accepted.

Wed, Feb 12, 10:04 PM · User-DannyS712, Spike, DannyS712-Global_watchlist.js
IKhitron added a comment to T245051: 104: Add the invocation link to all skins.

As you wish.

Wed, Feb 12, 7:40 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245051: 104: Add the invocation link to all skins.

Monobook and modern. Minerva has not the node either, but you already fixed it. Vector and timeless have the node.

Wed, Feb 12, 7:36 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron claimed T245051: 104: Add the invocation link to all skins.
Wed, Feb 12, 7:09 PM · User-DannyS712, DannyS712-Global_watchlist.js
Restricted Application reassigned T245051: 104: Add the invocation link to all skins from IKhitron to DannyS712.
Wed, Feb 12, 7:09 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245042: 103: Make the script RTL-compatible.

It is, in both versions. Thanks a lot.

Wed, Feb 12, 7:03 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245042: 103: Make the script RTL-compatible.

[[user:ערן]], [[user:ביקורת]], [[user:דוד שי]].

Wed, Feb 12, 6:14 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T245042: 103: Make the script RTL-compatible.

If you are talking about assigning, seems like the bot re-assignes it to you every time the task is edited.

Wed, Feb 12, 6:03 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron claimed T245042: 103: Make the script RTL-compatible.
Wed, Feb 12, 6:01 PM · User-DannyS712, DannyS712-Global_watchlist.js
Restricted Application reassigned T245042: 103: Make the script RTL-compatible from IKhitron to DannyS712.
Wed, Feb 12, 6:00 PM · User-DannyS712, DannyS712-Global_watchlist.js

Mon, Feb 10

IKhitron added a comment to T218511: After opening a diff, entry on Special:Watchlist sometimes stays unread (bold).

No, I'm seeing it constantly.

Mon, Feb 10, 5:07 PM · Regression, Performance-Team (Radar), MW-1.34-notes (1.34.0-wmf.5; 2019-05-14), User-notice, MediaWiki-Watchlist, Growth-Team

Sun, Feb 9

IKhitron added a comment to T244638: 101: show(Edits|LogEntries|NewPages) are not properly analyzed in getUserSettings.

Indeed. Thank you very much.

Sun, Feb 9, 7:36 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T244638: 101: show(Edits|LogEntries|NewPages) are not properly analyzed in getUserSettings.

Great, thanks. What about the stable version?

Sun, Feb 9, 7:24 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T244638: 101: show(Edits|LogEntries|NewPages) are not properly analyzed in getUserSettings.

Thanks, Done. Do not care if anyone will see my watchlists.

Sun, Feb 9, 4:57 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T244638: 101: show(Edits|LogEntries|NewPages) are not properly analyzed in getUserSettings.

I do not care how it will be done. Danny asked to create this paste.

Sun, Feb 9, 4:39 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T244638: 101: show(Edits|LogEntries|NewPages) are not properly analyzed in getUserSettings.

Access Denied: Restricted Edit Configuration
You do not have permission to view this object.

I've added some more debug logic. Can you please:

  • Open your global watchlist
  • Open the console
  • Run console.log( GlobalWatchlist.debugLog );
  • And create a paste with the result?

It'll be a large output, and (as a warning) will include what pages are being shown (generally watchlists are private; if you want me to create a paste only visible to the two of us I can do that too)

Sun, Feb 9, 3:23 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T244638: 101: show(Edits|LogEntries|NewPages) are not properly analyzed in getUserSettings.

Can you create a paste?

https://pastebin.com/UEXq1Nyk

Sure, but just leave it as disabled for now

No problem

Sun, Feb 9, 1:49 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T244638: 101: show(Edits|LogEntries|NewPages) are not properly analyzed in getUserSettings.

Done the first part. What to you want me to take after the refresh? I can still see the log entries.

Can you expand the ...?

It is already expanded.

What wiki are you viewing the watchlist on?

Ruwiki.

Can you post the whole console (again, expanding the arrays and objects)?

Yes, but are syre? It's about a megabyte of text.

Also, https://meta.wikimedia.org/wiki/User:IKhitron/global.js still says to show them...

It's because I restore it each time, for your next question. See the history.

Sun, Feb 9, 1:38 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T244638: 101: show(Edits|LogEntries|NewPages) are not properly analyzed in getUserSettings.

Done the first part. What to you want me to take after the refresh? I can still see the log entries.

Sun, Feb 9, 1:27 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron updated the task description for T244638: 101: show(Edits|LogEntries|NewPages) are not properly analyzed in getUserSettings.
Sun, Feb 9, 1:26 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron changed the status of T244638: 101: show(Edits|LogEntries|NewPages) are not properly analyzed in getUserSettings from Stalled to Open.
Sun, Feb 9, 1:12 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron updated the task description for T244638: 101: show(Edits|LogEntries|NewPages) are not properly analyzed in getUserSettings.
Sun, Feb 9, 1:12 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T218511: After opening a diff, entry on Special:Watchlist sometimes stays unread (bold).

Me either, three times last week.

Sun, Feb 9, 10:24 AM · Regression, Performance-Team (Radar), MW-1.34-notes (1.34.0-wmf.5; 2019-05-14), User-notice, MediaWiki-Watchlist, Growth-Team

Sat, Feb 8

IKhitron changed the status of T244638: 101: show(Edits|LogEntries|NewPages) are not properly analyzed in getUserSettings from Open to Stalled.
Sat, Feb 8, 1:40 PM · User-DannyS712, DannyS712-Global_watchlist.js
Restricted Application assigned T244638: 101: show(Edits|LogEntries|NewPages) are not properly analyzed in getUserSettings to DannyS712.
Sat, Feb 8, 1:40 PM · User-DannyS712, DannyS712-Global_watchlist.js

Tue, Feb 4

IKhitron awarded T187264: Pinging users from edit summary should send success/fail mention notification a Like token.
Tue, Feb 4, 11:46 PM · MediaWiki-Comment-backend, Growth-Team, Community-Wishlist-Survey-2017, Community-Tech, Notifications, Collaboration-Team-Triage

Mon, Feb 3

Restricted Application assigned T244134: Splite the script to four pages to DannyS712.
Mon, Feb 3, 2:05 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron awarded T244041: 99: Let users collapse each site results a Like token.
Mon, Feb 3, 1:40 PM · User-DannyS712, DannyS712-Global_watchlist.js
IKhitron added a comment to T243489: 97: Port GlobalWatchlist.js into a gadget [spike].

Actually, tried to do this and failed. Did not investigate what's happening, but my wild guess is that the script is too large to be a gadget. Do not know.

Mon, Feb 3, 1:40 PM · User-DannyS712, Spike, DannyS712-Global_watchlist.js

Jan 27 2020

IKhitron updated the task description for T243753: Vertical merging of table rows in Visual Editor may cause unwanted loss of a whole row.
Jan 27 2020, 12:37 PM · VisualEditor-Tables, VisualEditor
IKhitron added a comment to T243748: Merging table cells in Visual Editor destroys the content of all cells but one.

I'm not sure it was not intended.

Jan 27 2020, 10:36 AM · VisualEditor, VisualEditor-Tables

Jan 25 2020

IKhitron added a project to T243665: Hewiki: Allow patroller group to edit autopatrol protect level: Regression.
Jan 25 2020, 10:21 PM · Regression, User-Urbanecm, Wikimedia-Site-requests
IKhitron awarded T243665: Hewiki: Allow patroller group to edit autopatrol protect level a Burninate token.
Jan 25 2020, 8:37 PM · Regression, User-Urbanecm, Wikimedia-Site-requests

Jan 17 2020

IKhitron updated the task description for T243076: Change autoconfirmed days and edits limit in [email protected].
Jan 17 2020, 2:41 PM · Wikimedia-Site-requests

Jan 16 2020

IKhitron changed the status of T236476: [print] Avoid first section header alone on title page in "Download as PDF" feature from Stalled to Open.

Found it, @TheDJ. See here:


The H2 title "רקע" is on the last line of the first page.

Jan 16 2020, 2:17 PM · Browser-support-print-media, Readers-Web-Backlog, Product-Infrastructure-Team-Backlog, Proton, Collection, PDF-Rendering
IKhitron added a comment to T236476: [print] Avoid first section header alone on title page in "Download as PDF" feature.

Aklapper changed the task status from Open to Stalled.

Thanks. I thought to do it by myself, but was not sure.

Jan 16 2020, 1:57 PM · Browser-support-print-media, Readers-Web-Backlog, Product-Infrastructure-Team-Backlog, Proton, Collection, PDF-Rendering
IKhitron added a comment to T236476: [print] Avoid first section header alone on title page in "Download as PDF" feature.

It's a pity you're asking it only now. I do not remember where did I see it. I'll try.

Jan 16 2020, 1:54 PM · Browser-support-print-media, Readers-Web-Backlog, Product-Infrastructure-Team-Backlog, Proton, Collection, PDF-Rendering
IKhitron added a comment to T236475: Make links underlined in "Download as PDF" feature.

So you want the stylesheet for 'print' to be different for the PDF version, so that you can still recognise the links (correct?)

Jan 16 2020, 1:53 PM · Collection, Electron-PDFs, PDF-Rendering

Jan 7 2020

IKhitron added a comment to T237267: Decide the fate of <source>.

Once the patch is merged and deployed, and its easier to track usage, I'll file bot requests on the most affected wikis (and if there are less that eg 100 uses just replace manually) - can I state that it is now "official" that it will eventually be removed?

Jan 7 2020, 2:10 AM · User-DannyS712, Patch-For-Review, Parsing-Team, Technical-Debt, SyntaxHighlight

Jan 6 2020

IKhitron added a comment to T230103: Systematize wgRestrictionLevels.

Is there a way to get a list of renamings for a particular wiki?

Jan 6 2020, 11:26 AM · User-DannyS712, User-notice, User-Urbanecm, Wikimedia-Site-requests