Skip to main content
5 votes
Accepted

Retirement Calculator

Since you're using some ES6+ features, I'd definitely recommend looking into others, such as let, arrow functions, and so on. This website is a good start, and you ...
kingdaro's user avatar
  • 166
4 votes
Accepted

jQuery Accordion + Sub Accordion + Search Filter

Well, there is a room for improvement in a few areas, Save references to the elements rather than using them like $("#element_id") every time you need them to ...
Muhammad Omer Aslam's user avatar
3 votes
Accepted

Create multiple parametherized JQueryUI dialog boxes

Question responses What would be your method to improve the management of multiple functions? Looking at the three functions that call dialog_Handler() it appears ...
Sᴀᴍ Onᴇᴌᴀ's user avatar
3 votes
Accepted

Autocomplete selector country/region/city for ex-USSR countries

There is too much nesting, which, for me, makes the logic hard to read. A good rule of thumb is to aim for two levels of nesting, by using the Extract Method refactoring. When the user clicks on the ...
dcorking's user avatar
  • 321
2 votes
Accepted

JQuery fixed header that hides on scroll down and shows on scroll up

First improvement I saw is about that always true condition: if (st + $(window).height() < $(document).height()) The amount ...
Louys Patrice Bessette's user avatar
2 votes

jQuery Accordion + Sub Accordion + Search Filter

It seems like there is a defect with how the terms are searched. When you use a word that shows up in an answer and not the question, the answer is replaced in each subsequent answers. For example, ...
Coy W.'s user avatar
  • 21
2 votes
Accepted

Dynamically change data in Chart.js charts with jQuery slider

So, your stack is storing three different types of things: dates, totalBudgets, and ...
I wrestled a bear once.'s user avatar
2 votes

Autocomplete selector country/region/city for ex-USSR countries

dcorking has good points in his/her answer - especially about the redundant code (i.e. point #3). Below is my suggestion to improve the code, in addition to his/her advice. Cache DOM Lookups Like I ...
Sᴀᴍ Onᴇᴌᴀ's user avatar
2 votes

Retirement Calculator

Parsing integers The function convert() calls parseInt() with no second parameter (i.e. radix). The MDN documentation states ...
Sᴀᴍ Onᴇᴌᴀ's user avatar
1 vote
Accepted

Option buttons in plugin Dialog

Your Question Is there the better way to add a function to the option "buttons" to get the data and to go to a new dialog? I don't believe there is a "better" way, though instead ...
Sᴀᴍ Onᴇᴌᴀ's user avatar
1 vote

jQuery UI button and some calculations

Few things: The totalPrice variable should be declared inside your document.onready callback (and remember about ...
cezarypiatek's user avatar

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