Skip to main content
Share Your Experience: Take the 2024 Developer Survey
13 votes

Stack Exchange rev​iew queue notifications

I don't see much benefit to the enum. The Notification.permission string (or the return value of requestPermission()) is already ...
CertainPerformance's user avatar
11 votes
Accepted

Stack Exchange Chat Caret Pathfinder

There is still code-golf Here you have some golfed code: function(e){e.style.background = "";} ungolf it: ...
Downgoat's user avatar
  • 902
8 votes
Accepted

Make Stack Overflow Design Great Again

That's generally a really neat code you've got there. Here are my remarks, though: Applying styles and position of it in code Instead of applyCssToHTMLCollection()...
Przemek's user avatar
  • 1,212
7 votes

Stack Exchange Chat Caret Pathfinder

One line variable declaration Please avoid declaring variables on the same line: var elements = {}, messages; It's also a great way to royally mess up scope ...
Quill's user avatar
  • 11.9k
7 votes

Showing links to Contributor Breakdown on Stackoverflow Docs

You're using 'use strict'; in all functions except the top one. Why not declare it at the top of your file? You seem to have thought about readability. You're ...
Mast's user avatar
  • 13.4k
5 votes
Accepted

"Block a website" script with vanilla JavaScript

The main suggestion I have is to take the values from sitesToBlock and put them in the @match metadata key instead of detecting ...
Sᴀᴍ Onᴇᴌᴀ's user avatar
5 votes
Accepted

Colors for unvisted, visited and hovered/focused/active links

Your question My actual question is about 2nd and 3rd implementations. I don't sure which choose from them. Which is better? That might lead to some opinionated answers, especially because better ...
Sᴀᴍ Onᴇᴌᴀ's user avatar
5 votes
Accepted

SmokeDetector report notifier

A quote from another answer of mine: You don't use quotes or apostrophes for string literals consistently. Decide if you want to write string using either apostrophes or double quotes and use it ...
Michał Perłakowski's user avatar
5 votes
Accepted

Adding links to tags on StackExchange sites

Since Stack Exchange already uses jQuery, I think it's a good idea to write userscripts using jQuery too. However, rewriting the whole code to use jQuery instead of native APIs would require a lot of ...
Michał Perłakowski's user avatar
5 votes
Accepted

Userscript to tighten security in Mozilla Firefox

A few things: // comments don't need another // at the end, but you can leave it there if you want. You don't need to repeat <...
gcampbell's user avatar
  • 453
5 votes

Stack Exchange rev​iew queue notifications

Usability I ran the script on the SO review dashboard to see how it would perform with various numbers. The biggest thing I noticed was that it didn't properly handle numbers greater than 999 since ...
Sᴀᴍ Onᴇᴌᴀ's user avatar
4 votes

Adding links to tags on StackExchange sites

Style Your indentation is incorrect in a few places. At the start: (function() { 'use strict'; console.debug("BEGIN StackExchange Questions..."); You're ...
Quill's user avatar
  • 11.9k
4 votes
Accepted

Showing links to Contributor Breakdown on Stackoverflow Docs

No need for timeout, just attach in a place that doesn't get overwritten: after the title. Since the extracted link id number is always the last, we can process all links in one go, extracting section ...
wOxxOm's user avatar
  • 1,952
4 votes
Accepted

JSON photo data parsing

find the first "biggest" url size name. A more convenient way is using Array.prototype.find() method: p.id can be used directly as a key for object ...
RomanPerekhrest's user avatar
3 votes

Userscript to redirect insecure pages to HTTPS

Overview Pass global variables as parameters to IIFE to make your code more independent and clear Use addEventListener method to follow recommended way of ...
Stexxe's user avatar
  • 505
3 votes
Accepted

UserScript to Highlight tab for queue item

How does the es-6 usage look? What would you change and/or code differently? 1. Arrow Functions I would personally recommend using arrow functions where you have not to stay consistent with the ES6 ...
Grant Miller's user avatar
3 votes

Colors for unvisted, visited and hovered/focused/active links

Better how? In this case for me better is simple portable I have not spent too much time with User Scripts but I write front end code all the time, so I will do my best. Have you considered a User ...
TMB's user avatar
  • 209
3 votes

Poké-permalinks

I'd appreciate any feedback on improving perceived responsiveness for the feature this UserScript adds to the webpage. Since you are trying to synchronize the display of the image and the current URL ...
Grant Miller's user avatar
3 votes

Sort Facebook Messenger contacts by idle time

Normally I would suggest pulling up the functions. Though they get hoisted by JS, it's nice from a readability point of view that you know they exist first before code tries to use them. ...
Joseph's user avatar
  • 25.1k
2 votes
Accepted

Smoke the SmokeDetector

I'm sorry, but my review is going to be short; your code looks pretty good. Refactor node deletion You did a very good job of refactoring your code for the most part. The only repetition that I can ...
SirPython's user avatar
  • 13.3k
2 votes

Badge Oneboxer for Chat.SE

I apologize, but my review isn't going to be very vigorous. Don't check all elements Every time a new chat message is added, you are searching through all the chat messages on the page for messages ...
SirPython's user avatar
  • 13.3k
2 votes
Accepted

Onebox hider for Stack Exchange chat

For the most part it looks good, and there are some nice touches which I'll have to remember to look up the next time I write JS, but there are a couple of things which don't feel quite right. ...
Peter Taylor's user avatar
  • 24.1k
2 votes

Add timeline link to questions and answers on Stack Exchange

Direct per-attribute DOM building belongs to the '00s but, since you're using ES2015's let, let's use multiline template strings and make the final result of our ...
wOxxOm's user avatar
  • 1,952
2 votes

"Block a website" script with vanilla JavaScript

You can also explicitly run the script before the document loads by adding @run-at document-start to your userscript.
Dat Tran's user avatar
2 votes
Accepted

Font and color checker chrome extension in JavaScript

Just some random comments from a quick read through: 1) Does this work if someone defines their colors using hex or name style (#fff, ...
Marc Rohloff's user avatar
  • 3,467
2 votes

Tampermonkey script to beautify related tags in Stack Exchange

[...], but there is a variable that is being declared, used, then overwritten by the next line. Indeed. And it's not clear why it is that way. I believe you're talking about ...
janos's user avatar
  • 111k
2 votes
Accepted

Condensing a Table

Algorithm The algorithm to build the collapse-specs is inefficient: It makes a pass over all headers to find the percentage headers Then for each percentage header: It finds the index using ...
janos's user avatar
  • 111k
2 votes
Accepted

TamperMonkey script to add a button to Jira

The code is well commented. The indentation in most places seems to be four spaces but the last few lines have indentation of two spaces so that could be made more consistent. The last comment is a ...
Sᴀᴍ Onᴇᴌᴀ's user avatar
2 votes

UserScript to navigate to the next comic page on 8 Bit Theater by clicking on the panel on the current page

if(comicDiv){ What happens if you don't find the div? For example, if the site owners change their HTML so that's not the ID anymore? I suggest at minimum a ...
alexanderbird's user avatar
1 vote

Font and color checker chrome extension in JavaScript

General feedback It is difficult to tell exactly what this code is supposed to do, mostly because there are no samples of HTML where the code will and will not perform different tasks. I tried ...
Sᴀᴍ Onᴇᴌᴀ's user avatar

Only top scored, non community-wiki answers of a minimum length are eligible