13
votes
Stack Exchange review queue notifications
I don't see much benefit to the enum. The Notification.permission string (or the return value of requestPermission()) is already ...
- 8,733
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()...
- 1,232
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 is ...
- 29.6k
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 ...
- 29.6k
5
votes
Stack Exchange review 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 ...
- 29.6k
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 ...
- 5,337
3
votes
Accepted
Simple JavaScript Tampermonkey userscript to show absolute timestamp in Reddit
Kudos for writing a descriptive userscript doc comment, with a version (I don't bother doing this with my userscripts, but probably should).
For small userscripts, particularly small ones written for ...
- 4,197
3
votes
Accepted
Page scroll marker user script
Review
This seems to work acceptably and I don't have much to offer in terms of improving the styles or animation of the marker. I used to use Opera regularly in the past and vaguely remember the ...
- 29.6k
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 attaching ...
- 525
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 ...
- 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 ...
- 258
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 ...
- 258
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.
- 21
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, ...
- 3,547
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 ...
- 113k
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 ...
- 113k
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 ...
- 29.6k
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 ...
- 501
1
vote
Check if a website is in a certain language and if so, block it
Your script will only work correctly if it is placed after the Yod character. Consider adding an event listener for window.load to only call it when all the content ...
- 2,467
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 ...
- 29.6k
1
vote
Altering SE sites to imitate SO's appearance
Feedback
This is an interesting script. I understand the desire to update the styles but don't see the point in changing the titles and placeholder texts to contain Stack OverFlow since it isn't the ...
- 29.6k
1
vote
Code-Q2A - copy code blocks from questions to answers
I know this code has been around for ~5 years but still appears to work properly despite some DOM changes on these sites. You no doubt have learned a lot since posting this but I feel there are other ...
- 29.6k
Only top scored, non community-wiki answers of a minimum length are eligible