« first day (388 days earlier)      last day (1051 days later) » 
00:00 - 14:0014:00 - 23:00

12:02 AM
@Scratte only 33 problems
^ total, both files (same as before - most depend on proper types missing)
a bunch - because by default, TS does not know what Svg library from SE is
I wonder if there are types for it
phew
 
12:27 AM
btw, @Scratte, another argument in favour of not using "Yes"/"No":
when you need a boolean, you simply do something like this:
checkbox.checked = option;
instead of checkbox.checked = (option === "Yes") ? true : false; (which itself should be checkbox.checked = (option === "Yes") because the comparison is already boolean)
 
 
4 hours later…
4:34 AM
@oleg problem solved, I had to login with a PAT, not my GH password
 
4:58 AM
@double-beep huh, glad it is solved for you - I wish this (I suppose you encountered the same issue as described here) was better documented
 
5:42 AM
@OlegValter the AutoReviewComments is script is abandoned - do you think we could create a clone?
 
@double-beep hm... Is this the one? I suppose it is
 
yes, that one
 
@double-beep thanks for confirming - I see there are 36 forks already, I am not sure if anyone else is working on it currently. If there are any repos that are still alive, maybe it will be better to collab? Otherwise, yeah, let's fork it and shape up (let me take a quick look through the list)
 
5:58 AM
https://github.com/machavity/SE-AutoReviewComments - Jan 2021
https://github.com/SmartManoj/SE-AutoReviewComments/tree/patch-1 - Oct 2020
https://github.com/codygray/SE-AutoReviewComments/tree/master - Aug 2020
https://github.com/CertainPerformance/SE-AutoReviewComments/tree/master - May 2020
well, yeah, that's the list of most recent updates
I suppose Machavity can still be interested? Not sure
@double-beep do you propose a full rework or a fork + go from there?
 
@OlegValter Machavity announced the creation of his fork in SOCVR I think
I think a fork would be better
we can then erase the unneded files
and rewrite the whole thing in TS
 
@double-beep hm, I like that :)
@double-beep fork it is then, sounds good to me
@double-beep unneeded files? ok, let me take a look at the repo, maybe I'll get it then
 
the extension-related files
 
6:15 AM
@double-beep back from the repo :) Figured - yup, so, I suppose, we just want to take the userscript part?
come to think of it, if we do not need the extensions, do we really need a fork? I am too lazy to copy-paste the CI part already configured for the template repository :)
 
6:47 AM
I think it would be the best way to attribute to the original
 
@double-beep yeah, you are right, it's my 10 AM after a sleepless night speaking - I'll go ahead and fork it
 
7:07 AM
@oleg we also probably need to enable issues
and I can't do that
 
@double-beep huh? oh! done!
I've created a cleanup branch with a couple of initial setups like CI and testing base frame, nothing else as of yet, even the TS config is not configured other than for enabling types
 
7:33 AM
@OlegValter But at least I only have to update that in one place now :)
@OlegValter That's a lot less than I would have expected, since you didn't code review this one.
@OlegValter Please go and sleep, so you don't become pale and transparent :)
Ohh wow, you're doing autoReviewComments? :)
 
@Scratte yeah, @double-beep's idea :)
@Scratte that's just a setup :)
 
I've been wondering if I could hard-code some comments into my version of the script.
It works if I put in stuff that goes with "[A] Affiliation ..."
 
@Scratte hard... code... comments... ??
 
But it doesn't if there's no "[A]".. so if I want them to appear on both types of post, it doesn't work
@OlegValter Do you have it installed?
 
@Scratte nope, obviously not
 
7:45 AM
@OlegValter Can you run it?
 
@Scratte sure
 
So when you open it, there's an import/export feature on the bottom.
Then you can replace the entire content in there. Which I do every time I repload my browser.
Because I have one that looks like this:
### Signatures
Please don't use any kind of [signature, taglines, or greetings](https://$SITEURL$/help/behavior).
It doesn't have a type.. it's just a general comment.
As opposed to:
###[Q] Duplicate Closure
This question will probably be closed as a duplicate soon. If the answers from the duplicates don't fully address your question please edit it to include why and flag this for re-opening. Thanks!
This has a [Q] type..
That one above is a hard-coded one that comes with the script, so when it's first installed, that will always be one of the options.
 
ah those, I think I want to get rid of the hardcoded ones entirely
 
Argh!.. I was hoping I didn't have to load them in every time I open up my browser :(
 
@Scratte 🤦‍♂️
 
7:51 AM
It repetitive work.. I'd rather have my comments in the script.
 
not in the sense that no default ones but I'd like to think if they can be moved out of the source
 
the script probably needs to use GM_* functions
 
at least they need a liftup
 
@double-beep Not sure any of that helps. I use my browser incognito. I'd like to modify the script and have my comments in there in the version that is on my system.
 
prob'ly can't, though
 
7:52 AM
@Scratte doesn't TM's storage persist even in incognito?
@OlegValter what if we check if GM_* exist and if not use localStorage?
 
@double-beep Does it matter?
 
that would make everyone happy :)
 
OK. I'd still like to find out how I can put those in the source.
The source for some reason requires a type. I have 5-6 comments that do not have a type.
 
@double-beep guess that's a good idea (I will still try to accommodate my use case of not liking script managers too much, so a fallback to localStorage sounds unavoidable)
 
And I'd like to not put them into localStorage nor into extension storage, because they are not settings. They are my personal crafted comments.
 
7:56 AM
you'll have to do the same every time the script is updated
 
@Scratte not sure I get you?
 
@OlegValter If I delete the script, will my comments still be in the storage?
There are two option. Put them into storage that is kept through sessions will solve most user's issues since they don't use incognito. Use the extension storage, which will keep them as long as the extension is still there.
Or 3. Just edit them into the source-script on my local system, which is what I tried to do, but hit a wall with the no-type ones.
And yes. I'll have to do it every time the script is updated. I have no problems with that. As long as it's not broken, I don't update.. :)
 
@double-beep does it? That would be nice. I can try and test later today, though
I suppose it can be using chrome.storage API (if we are talking about the Chrome extension that is) which is persisted even across incognito
 
But is that preferable?
 
8:11 AM
maybe it's my tiredness but I lost track why we are discussing this...
I certainly do not like arbitrary comments in source in the repo apart from those that are already there historically
and even those I am not sure about
so... @Scratte if you intend to edit the code for yourself I do not see the problem
unless you want to make your comments included for every user that installs the userscript
 
@OlegValter I do want to edit the code for myself. What I am saying is that the way the code is currently structured made it impossible for me to add my comments to the code.
 
@Scratte I think we've been talking about very different things all this time then :)
ok, so
 
Because inside the code, it expects the "defaultcomments" to have a type.
@OlegValter Yes. I think may have been talking about two different things :)
 
@Scratte type? You mean "target", right?
 
@OlegValter Yes, I expect that's the "target"
But I was less experinced when I tried to do this.. so perhaps I'll have fewer issues with it now.
 
8:18 AM
but when I look at the source I see that the target is used only to create a prefix
 
I just think it would be really nice to have this option that people can just edit in their own comments directly into the script and not rely on a @requires directive or localStorage or something else.
 
oh, scrap the above - I finally have a general grasp
 
So.. now I just tried to add this:
 
@Scratte but they always can do that, no?
 
  {
    Target: [],
    Name: "Test",
    Description: "...."
  },
That comment doesn't show :)
@OlegValter That depends on the level of commitment that is required to make it work. When the code is very convoluted or users face a wall, like I did.. then I do not consider it a real option.
I remember when I tried this. I kept logging in and out of Stack and reloading my browser to see if my changes had taken effect. After a few days of doing this on and off.. I just gave up.
And now I import/export my file in every time I log into Stack.
 
8:24 AM
@Scratte well, as currently written, it needs a target, I think. Why not Target.MATCH_ALL since the comment is general-purpose? It will show up just fine
 
I can try that.. next week when I reload my browser again :D
I'm pretty sure I tried it already though
 
@Scratte I am sorry to say this, but I do not think yours counts as a normal use case :)
that said, this should work
 
I know my case isn't the normal use case :)
My reason for using a user scripts is to make custom modifications.. :P
 
@Scratte ahem, why do you need to reload the browser to do that?
 
@OlegValter I think it only looks at the defaultComments when there are none in storage.
I hadn't thought of forcing it to always empty the storage.. or even change the way that that works.
 
8:31 AM
@Scratte I think you are really overthinking this :)
so, why not do this:
1. go to import/export and copy everything
2. hit the "reset" button
3. paste custom back in and save
step 0 is editing the source
 
Heh.. yes, thanks :) Definitely overthinking it :)
 
@Scratte I even went ahead and confirmed that myself :) Not that there is anything that could go wrong, though. So, where's the problem again? :)
 
I see that brings down the calendar time I need to find out how to do this :)
I think it doesn't use the match_all to match :)
I'll debug it properly at some later point. Thank you for telling me about "reset". I hadn't noticed that at all :)
 
@Scratte maybe because all of the control buttons are grouped to a tiny line at the bottom? :) The UI needs a liftup definitely
 
Maybe so, yes. The whole "put my comments into this script" is just a huge item on my list of this to do that's been annoying me for a long time :)
I've procrastinating it, because I know it'll take me a while..
 
8:42 AM
@Scratte but... it's like 5-minute task tops?
 
@OlegValter Not for me, it's not.. and still not.
 
@Scratte I am unsure why you can't just dump the comments you need matching the structure of the elements of the array, give each Target.MATCH_ALL (if they are versatile) and be done with it?
 
  {
    Target: [Target.MATCH_ALL],
    Name: "Test",
    Description: "...."
  },
 
I know :)
you can also not give it any target
and hit reset - that's the important part
it will show up as a pair of empty brackets
 
I just tried all kinds of different ways to do it. Then I put a note in my Todo list.. "Study and debug this script"
@OlegValter But I don't want a pair of empty brackets :)
I'd like it to look exactly like it looks when I import it ;)
 
8:46 AM
@Scratte it's UI only - try inserting it :) But that's exactly what we could improve - assuming comments can have no target and by default be counted as versatile
oh, I think we do not even need to do that
god damn it
misclicked
"just omit the Target property altogether"
 
@OlegValter :O That seems to work!
I think if this was my code, I'd put a note there to anyone that wants to edit them :)
I can't believe that worked! :) I'm so happy now! :)
You're my hero! :)
 
@Scratte you are probably the only one who thinks that :)
it's pretty obvious once you read the source carefully
 
Perhaps.. perhaps I'm not the only one that makes some small custom modifications to the source :)
 
I think that should've simply been mentioned in the documentation (maybe it is, I am yet to read up on ARC)
because it is equivalent to adding a ###Header in the GUI
 
@OlegValter Heh.. I think you're better at that.
 
8:54 AM
@Scratte no, I am surprised it took me that long to notice :)
@double-beep - will you support a non-jQuery rewrite, btw?
 
@OlegValter Heh.. I'm still trying to read it :)
 
I'm not a huge fan jQuery either
 
@double-beep great :)
 
I've also opened a PR where we can discuss the changes easily :)
 
@double-beep noticed, thanks :) should it live as a draft for a while? I guess there is no point in not finishing the tsconfig before merging, for example
 
9:04 AM
I thought we could use the same for the rewrite, too
 
@double-beep ah, ok, no problem - I thought you wanted to make multiple PRs towards the 1.5.0 milestone since you assigned the PR to it
 
9:23 AM
all right, TS config's done (@double-beep I am not sure about emitting declaration files - I guess it's not really needed for a userscript?), methinks
ugh, the rewrite's gonna be tough
 
9:49 AM
scratch what I said earlier, we're certainly going to need a new branch
you can go ahead and create rewrite and then a PR pointing to master
 
@double-beep huh, why such a sudden change? :) Btw, I think we all should get a good grasp on who does what, methinks. I already started moving to TS, I think I'll manage what I can, but changing from jQuery everywhere is gonna kill me, will need help
@double-beep yup, will do just that
 
the first step would be to get rid of all the vars
 
btw, now I have to ask a probably silly question - does the StackExchange global object have typings somewhere out there?
@double-beep actually, the first step is to deal with 112 (and I already gone down a 40) issues reported :)
 
I'd help.. but I think my "help" is going to be extra work for you :(
 
@OlegValter there aren't, but I can help, since I've written some for one of my scripts
 
10:00 AM
@Scratte moving jQuery to simple DOM manipulation? :) I don't think it will hurt
@double-beep oh, thanks, if you have a declaration it would be great - it's certainly better than me retyping the whole thing :)
 
@Scratte you must be the only person in Denmark unaware of what happened to Christian Eriksen last week. Anyway, this will be in the news daily for the next couple of weeks. (And contrary to your commentary, it's not only "rich people" who take pain-killers and anti-inflammatorys.)
 
these are written by me and these by Rob
 
@double-beep oh, that's very very nice, thank you!
 
@bad_coder Probably not the only one :) But I was completely unaware of it. I'd assume they use them more.
 
I think we can publish a full definition separately later on and reuse
 
11:09 AM
Hello. :)
Why we have forked Auto Review Comments userscript?
 
@KevinM.Mansour Read the transcript..
 
@Scratte Huh.. The transcript have more than thousands of message for 14 June 2021. :)
 
@KevinM.Mansour So, you think that someone should repeat what has been said instead of you pasting in "Auto Review", "Auto Review Comments", "AutoReviewComments" in chat search on the top right and see what turns up? :P
 
@Scratte You are so helpful. :)
 
For someone to find the beginning of that conversion, that someone would have to paste in those keywords in search anyway.. so it might as well be the one that wants to know, no?
 
11:19 AM
1
Q: Alignment issue in the "Top Questions" page

ArulkumarIn the Top Questions page, the asked/modified X mins ago text was left-aligned next to the tags. Previously the text was right-aligned. Seems some alignment issue introduced on that page recently. Screenshot for reference: Configuration Version Operating System Windows 10 Pro Mozilla F...

 
LOL!. It got worse
 
There is lot of bugs in Stack Exchange.
 
Didn't someone post on meta about askers not being on the right side? in the /questions page?
I bet the new issue is a result of trying to fix that one ^ :D
 
@Scratte To fix Issue, They produced new issue. :)
@Scratte You are so funny.
 
Apologies for the miscommunication here, folks: I guess Rosie and I missed that the fact that the user card is not anchoring to the right was the problem. We focused on the new line bit of it. In any case, it looks like someone already pushed a fix for this before we'd even triaged it ;) It'll be live in the next build — JNat ♦ 2 days ago
.. so, I going to take a wild guess and say there may just be a connection there :)
Heh.. no.. both bugs are live on Stack Overflow. Bug #1 and Bug #2
 
Seems they added float: left; to the ".tags" class.
 
@Scratte In questions page?
 
they added the tags class, that's the issue
$('.question-summary .tags').removeClass('tags');
 
@double-beep Hmm.. sure, but perhaps that class needs to go on the tags, no?
 
they should rewrite the whole thing using an appropriate Stacks component, e.g. post summary
 
11:49 AM
@double-beep That first examples wraps around the user ;)
 
In .gs4 > .grid, .gs4 > .grid--cell remove margin:2px; then go to .post-tag and remove font-size:12px;. That will fix problem.
I will develop userscript. :)
 
I think that will cause some other problems.
I'd like to keep the margin on the "grid-cell" common class.
What issue is this fixing?
 
@Scratte I don't see it will cause any problems.
@Scratte Questions Page.
 
@KevinM.Mansour Will that not just make stuff smaller?
And then when there is more text, the problem will just show again?
 
@Scratte No. It will fix problem. it will look like others. Just Try it.
 
11:56 AM
Can you link to a page where you say that this will be fixed?
 
[Questions tagged [huawei-mobile-services]](stackoverflow.com/questions/tagged/huawei-mobile-services)
 
@KevinM.Mansour hmm.. "Questions tagged [huawei-mobile-services]"
 
Try on the first question. You will see it is not aligned correctly.
@Scratte What does that mean?
 
Please put nice title on such, so what other users that search chat can find it again.
Ot perhaps work
 
@Scratte I tried but it didn't work.
[Questions tagged [huawei-mobile-services]\](stackoverflow.com/questions/…)
 
12:00 PM
You need to escape the [ and ] with a \
Huh?.. that looks like two backticks.
[Questions tagged \[huawei-mobile-services\]](https://stackoverflow.com/questions/tagged/huawei-mobile-services)
 
You can't use a ] in the toilet-link, because a ] means the end of the title. So you need to escape it.
 
@Scratte Thank you. :)
 
I can't see a ".gs4 > .grid, .gs4 > .grid--cell". Which is this element?
^ that seems to be a completely unrelated marketing link
 
@Scratte <a href="/questions/tagged/xamarin.forms" class="post-tag grid--cell" title="show questions tagged 'xamarin.forms'" rel="tag">xamarin.forms</a>
On The Tag Itself. :)
 
12:08 PM
That's a different element, no?
I have no matches for "xamarin.forms"
 
@Scratte Also .post-tag is on the same element.
 
Wait.. now I found it.. and it's not wrapped around
 
@Scratte Huh.. Inspect the tags.
Hope that helps.
Also you might keep font-size:12px;.
 
I see.. now go and find the element and look at the grid with the margin: 2px
 
At it will make things smaller but remove margin:2px;. It just create white space around the elements which makes user card to get down.
 
12:14 PM
Now.. go up so that you can see your "watched tags" on the same page. And toggle that little checkbox.
 
@Scratte What do you mean?
 
When I remove that margin, the margin is removed on all elements :) Including all the elements in the "watched tags"
 
@Scratte Huh? Because it is global class used everywhere.
But they looks fine anyway.
 
But your solution is still not going to help if there is 5 tags with 30 characters in each. It will only fix it for this one.
@KevinM.Mansour I think your solution needs to be re-evaluated. It would probably to better to separate the user card out, so that it never wraps around, instead of relying on lesser space and smaller font.
 
@Scratte Hmm? Good :)
 
12:17 PM
I'm sure you'd also see the effect on every post, your profile and everywhere else if you change a common element that that :)
 
I have got another idea :)
 
Of course.. you can change it for only this one thing if you select the element with a list of classes in proper sequence.. but I'm not liking the removal of the small space between the tags.
 
Go to div element have class grid ai-start jc-space-between fw-wrap. then remove fw-wrap and it will fix problem.
 
Can you say which is the element on the page? I mean what is the element, that I see?
 
I have added screenshot. :)
It is just an element. Inspect tags then scroll above a bit and you will see same like screenshot.
 
12:24 PM
Hmm.. for me to find out what element you mean, I have to do a querySelector("..") then "edit as HTML".. and then it will tell me which element that I am modifying.
I assume you know if it is the tag or the user card or the container that holds the tags, no? :9
 
I am not joking. :)
For me it fixes the problem.
 
Does it?
Are the user cards aligned between posts?
 
Yes, It does.
 
@Scratte Did you remove that class?
 
12:33 PM
and here is another long tagname.. Oops! The user is gone :D
 
@Scratte Did you remove that class?
 
Try to see it. Do you not see that first thing? Where is your screen shot with the class gone?
That is my screen shot with the class gone.
 
Hmm.. I think you did more than just remove the class then. But the user cards are not aligned.
The first one is way over to the right compared to the post under it.
 
@Scratte No, I didn't I just removed the class.
 
12:39 PM
OK. I think we need another way to do this.. because trying to do what you tell me to do, and show you how I don't think it's a good solution is not very fruitful. Especially not when you then do something else and then show me how that worked.
 
@Scratte You don't listen to what I say. I told you to remove that class and it fixes the problem like in the screenshot above but you didn't do that and just trying to be funny Ops! User card is gone and that is not funny and won't help and you just don't do what I tell; You just skip it and I asked Did you remove that class but you didn't answer and when I remove that class not thing else is affected anyway.
 
I did exactly as you said. I removed the class.
It's the only thing I did.
 
@Scratte Paste a screenshot with DevTools.
 
And I noticed that it went way to the right.
Why? I see the same in your screenshot?
 
@Scratte I do not understand.
 
12:50 PM
If I remove the fw-wrap class from the page, then my screen shot is what it will look like.
If I only remove it from the container that holds both the user cards and the tags, then the tags will wrap around.
But.. do we want the tags to wrap around?
 
@Scratte Which screenshot you mean?
 
And do you went the user card to move further to the right?
How is this suppose to look like when it's "fixed"?
this no class on the entire elements.
Anyway.. I need to fix something in my flat.
 
@Scratte I don't understand anyway. I will let you find better solution anyway.
@Scratte Good Luck :)
 
@OlegValter still working on AutoReviewComments?
 
@KevinM.Mansour I think it should be like this ..aligned to the same vertical line.
 
12:57 PM
@Scratte If I found a better solution, I will tell you. :)
Algined in same vertical and fix usercard.
 
I think perhaps Stack will try to fix this..
I'm almost sure they will try to fix the front page.
 
@Scratte In next build.
 
I can wait for that :)
I see I'm not the only one that thinks there is a connection:
There was this other, pretty related issue recently. The persistence of this kind of issues may suggest at a fragmented, hard-to-manage codebase: to me it seems like a proper overhaul and tidy-up of this component could be beneficial? While working as a dev, I used to like these tasks: hunting down pieces of code related to a component scattered around here and there, and unifying them in a tidied-up, canonical component definition... (Such refactors and tidy-ups are a kind of repaying technical debt: a normal part of developing any codebase.) — Levente 14 mins ago
@OlegValter I have a system with my comments :)
Adding spaces to make them aligned means I will never remove those.. I will not notice them in a clean up task.
I understand that you want some sort of rule.. but there a diff on every line :(
..and the diff engine is so confused about it that they don't line up.
Example:
This line will stay in my code:
` // console.log({query});`
This line will be cleaned up before I release it:
// console.log({query});
Maybe.. we should talk about some of the rules :)
Because it will take me a very long time to find out what changes are actual changes and what changes are just messing with the spacing.
But if you have a check going on that changes spacing, can you please just only run that on a commit?
And then if you think code should change, do that in another commit? So that it's obvious what really changed?
I know you feel strongly about the true/false.. but I'm not in agreement with it.
This also includes the new spacing rules.
 
1:18 PM
What does the // @namespace mean in the userscripts?
 
Example of what makes sense to me:
functionCall(param1,
             param2,
             param3);
This doesn't make a lot of sense to me:
functionCall(param1,
    param2,
    param3);
 
@Scratte That more clear in aligning.
 
This is a bit of the code after the re-indenting:
modalContent.append(messageToggle,
    messageColour,
    messageSize,
    messageBackgroundColour,
    previewMessage);
Which confused me when I scan the code.
I think perhaps this:
    if (fontSize)      label.style.fontSize   = options.fontSize;
    if (fontWeight)    label.style.fontWeight = fontWeight;
    if (width)         label.style.width      = width;
    if (indents)       label.style.marginLeft = (indents * 15) + "px";
    if (alignIndents)  label.style.marginLeft = `${(alignIndents * 15) + 10 + 44}px`;
transformed into this:
    if (fontSize) label.style.fontSize = options.fontSize;
    if (fontWeight) label.style.fontWeight = fontWeight;
    if (width) label.style.width = width;
    if (indents) label.style.marginLeft = (indents * 15) + "px";
    if (alignIndents) label.style.marginLeft = `${(alignIndents * 15) + 10 + 44}px`;
is a mistake.
 
@Scratte Clear.
 
I'm sure it's not a conventions, but:
const { colours: { seperator : seperatorColour } } = modalConfig;
That space before the colon in "separator" means I changed the name.
I know that the spacing here is part of the convention:
    if (!firstElementChild
        || !infoTextNode
        || infoTextNode.nodeType !== Node.TEXT_NODE
        || infoTextNode.nodeValue.trim() === "")
        return;
But that is very confusing to me. The || is part of the if-check.
Which is why I add in two extra spaces, so it becomes:
    if (!firstElementChild
          || !infoTextNode
          || infoTextNode.nodeType !== Node.TEXT_NODE
          || infoTextNode.nodeValue.trim() === "")
        return;
 
1:40 PM
@double-beep had to stop for a while - I have a lot of work to do, and I haven't slept in ~22 hours straight :(
 
@OlegValter I prefer to get a break for hours. and when you wake, you can return to your work. :)
 
00:00 - 14:0014:00 - 23:00

« first day (388 days earlier)      last day (1051 days later) »