11 votes

Semantic markup for card blocks

200_success' comment is right, that the context and the content of the component is important for semantics. I assume, from the linked page, that you want to display a list of short articles that lead ...
insertusernamehere's user avatar
9 votes
Accepted

Karma Sutra game

Adding feedback in the form of hints like Select the first position, then Now select the second position would enhance UX. Now ...
wOxxOm's user avatar
  • 1,952
7 votes
Accepted

Create a dynamic growing pyramid

A few thoughts on the task and your solution: Limited to 5 elements You're limiting the pyramid to 5 elements. The task doesn't have a restriction like that. I see two possibilities here: Do not ...
insertusernamehere's user avatar
7 votes
Accepted

Simple HTML website with JavaScript navbar

General disclaimer: You may notice that in one section I recommend not using something in this case, while in the next paragraph I show how you can improve the part I recommended not to use. I mainly ...
Sumurai8's user avatar
  • 2,595
6 votes

First HTML5 game: Snake

You did really well with readability and documentation. On the design side, I would personally have the score off to the side and not in the middle, although it isn't that big of a deal. I would also ...
Confettimaker's user avatar
6 votes

Simple website HTML

This is my personal advice. When it comes do ID's and Classes, ID's should only be used on things that will ever be used once, this can be hard to know if something only will be used once since a lot ...
Johan Gudmundsson's user avatar
6 votes
Accepted

Load JSON from server using jQuery and display it in html5

Some ideas: #1- I suggest using .map().join() or even better .reduce() which is more ...
Sergio's user avatar
  • 353
5 votes

HTML5 sectioning for my design

My review will be nowhere complete. Here are some points I would change: With a doctype of <!DOCTYPE html> all browsers will recognize the document as HTML5. ...
Pyfisch's user avatar
  • 661
5 votes

A simple Javascript Breakout game

Code review Not bad, code is nice and clean, and naming is good. Game play has some logic flaws in the way you handle the keyboard and collisions. Some code style points. Use constants for ...
Blindman67's user avatar
  • 21.8k
5 votes
Accepted

Console.log challenge change all of the links on a search results page so that they do not go anywhere?

Refactoring is commonly seen as "moving out repeating code into reusable functions". But it can also easily mean "moving code that's only ever used once back to where it's being used". Although the ...
Joseph's user avatar
  • 25k
5 votes
Accepted

Example of responsive webpage

Simplifying overall layout While I haven't gone through the coursera videos I did look at the syllabus. I see that the next assignment does involve using Bootstrap framework. Since this assignment ...
Sᴀᴍ Onᴇᴌᴀ's user avatar
4 votes

JavaScript for toggling appearance

Moar built-ins... MOAR (and a look-up table)! It is great that you are using the built-in insertRule method for style sheets here: ...
SirPython's user avatar
  • 13.3k
4 votes
Accepted

JavaScript function for to toggle multiple CSS-classes at once

This code is well-done, clear and rigourous, and it works fine. But I would not work exactly the same. Error processing. I agree with your doubts, like this one: I also ask myself if my approach ...
cFreed's user avatar
  • 2,811
4 votes
Accepted

Wikipedia Viewer

Here, it looks like you are trying to go around the async behaviour of AJAX: ...
ᔕᖺᘎᕊ's user avatar
4 votes
Accepted

Basic HTML structure for three columns of articles

HTML is what's known as a markup language, which means the actual meaning of the elements (whether you choose div or sections) ...
Quill's user avatar
  • 11.9k
4 votes
Accepted

Simple landing page markup with basic HTML

Overall this is quite good, by far not the worst HTML I have seen. I like that you use alt attributes on all your images, as you should. I am going to assume that ...
James Fenwick's user avatar
4 votes
Accepted

Structuring basic client side / serverside email form

Generally, I think this code looks really good for a pretty basic example of web form handling, particularly for someone who may be earlier in learning process. I think you have really easy to read ...
Mike Brant's user avatar
  • 9,608
4 votes
Accepted

My grade calculator with special weighting

A couple of minor things: fog and ing aren't very meaningful id names, they would be better as something like ...
Bethan's user avatar
  • 149
4 votes

JavaScript live grade Calculator

Firstly, there's an error in the code you've written which will prevent the calculator from working correctly: ...
James Hay's user avatar
  • 141
4 votes

A program to tell you if there are 2 numbers in an array that add to a given sum

There are a few problems with the function right now. gotPair leaks variables to the global scope. When you say this.data, <...
Gerrit0's user avatar
  • 3,456
4 votes

A program to tell you if there are 2 numbers in an array that add to a given sum

Danger code! Did you do a good job? Well I could not give your function a pass. Your solution works, but I would never use it in any environment apart from the testing environment. The reason is your ...
Blindman67's user avatar
  • 21.8k
4 votes
Accepted

Make HTML/Javascript Geolocation function more linear

Most native functions require callback instead of Promise because they support older browsers, so what you have to do is to pack the ...
Pavlo's user avatar
  • 408
4 votes

map of SF transit cars

Just as Gerardo hinted at in comments, I admit that I am not well-versed with d3 and at most can offer general JavaScript feedback. Nonetheless here is what I came up with: Since you are using ES-6 ...
Sᴀᴍ Onᴇᴌᴀ's user avatar
4 votes
Accepted

Dynamic country, state select options

There are 4 different places in the code, where you do something for every country. That makes it complicated to add more countries, and also makes the code very long if you have many. But you can get ...
Kruga's user avatar
  • 1,124
4 votes
Accepted

Calculator app in React

As a preface: Your code works good, is readable and does its job efficiently. The following review, though long, doesn't mean there is anything substantially wrong with it :-) General comments In the ...
gitcdn's user avatar
  • 431
4 votes

Simple HTML5 registration form

It would help Google and other crawlers if the <head> also included: ...
Ray Butterworth's user avatar
4 votes

Data selection and filtering

Indent your code! Naming convention Only use English paivitys -> update -> though really this should be with a subject, so updateObjekt? dataa -> data arvvoja -> ...
konijn's user avatar
  • 33.2k
4 votes
Accepted

Semantic implementation of the structure of a web page with header, main, and footer

I myself am also a beginner in the world of web development (So any comments about my answer are more than welcome!) But maybe my tips can still be useful in one way or another. ;-} The first thing ...
Valeria's user avatar
  • 56
4 votes
Accepted

PHP Class to render HTML div styled tables v1

I'm not overly bothered by the amount of arguments in your __construct() method. It looks fine to me. There are however some classic coding problems: Repetition ...
KIKO Software's user avatar

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