Skip to main content
Share Your Experience: Take the 2024 Developer Survey
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
5 votes
Accepted

Building a multi-panel display with React and Bootstrap

Move variables out of render All the variables you are storing are sitting in the render method of the respective component. ...
SirPython's user avatar
  • 13.3k
5 votes
Accepted

Parking Lot Management System in JavaScript

Prefer const over let and var. When you declare a variable with ...
CertainPerformance's user avatar
4 votes
Accepted

My minesweeper implementation in Javascript

Here are my personal thoughts: Naming: it's probably better to have placeBombs instead of initBombs, because you have a ...
ChatterOne's user avatar
  • 2,805
4 votes

Single page site with navigation bar

You should add <meta http-equiv="X-UA-Compatible" content="IE=edge"> <meta name="viewport" content="width=device-width, initial-scale=1"> to your <...
Denis Priebe's user avatar
4 votes
Accepted

Display and search through a JSON's items in Vue.js

Overall the code looks fine. There aren't any drastic changes I would advise - just mostly a few simplifications. As far as the design goes it looks good, though it may be wise to add a spinner icon ...
Sᴀᴍ Onᴇᴌᴀ's user avatar
3 votes

HTML/bootstrap demo page

Naming You should start naming your classes something a little more readable than div1 as all that tells me that it's a div but nothing regarding functionality. ...
Neil's user avatar
  • 421
3 votes

Bootstrap file input styler

Function('return this')() It took two minutes to me to understand what it does. Let's see what it does step by step ...
Tushar's user avatar
  • 3,002
3 votes

PHP contact form using PHPMailer and Google Recaptcha

Huge problem You have 1 HUGE problem in your code: If you enter correct captcha - the form is sent. Doesn't matter if it passed validation or not. It is a logic error in your post processing. You ...
Anatoliy Kim's user avatar
3 votes
Accepted

HTML5 markup for blog article

Just scratching the surface of the html code. Bear in mind it's difficult to review html since we are not focusing on the CSS and Javascript sections . Since am a Bootstrap fan most of my comments are ...
Tolani's user avatar
  • 2,465
3 votes
Accepted

CSS Grid and Bootstrap 4

Semantics and UX You can adjust a few things that will improve the overall UX of your login-page: Document outline / structure To improve the overall structure of your document use sectioning ...
insertusernamehere's user avatar
3 votes
Accepted

Add or remove a class on two elements when two different events fire

Don't repeat the queries. Store any queries that are called more than once in a variable, and utilize jQuery's chaining: ...
RoToRa's user avatar
  • 11k
2 votes
Accepted

Forum project using OOP PHP

back: I'm not sure what this function is used for (and the name doesn't seem that great), but it is open to XSS. An attacker could create a page at ...
tim's user avatar
  • 25.1k
2 votes
Accepted

Simple JavaScript quiz application with radio buttons

Object to class Each object inside your allQuestions array follows the exact same "format": ...
SirPython's user avatar
  • 13.3k
2 votes
Accepted

PHP contact form using PHPMailer and Google Recaptcha

<?php require 'assets/library/PHPMailer-master/PHPMailerAutoload.php'; require 'assets/library/recaptcha-master/src/autoload.php'; I am guessing that this ...
Mike Brant's user avatar
  • 9,608
2 votes

Excerpt from a grocery shopping site using SASS

Just few things to highlight here Misusing Twitter bootstrap: The grid structure in bootstrap allows up to 12 columns across the page. ...
Tolani's user avatar
  • 2,465
2 votes
Accepted

Interview project to redesign a website for a property

For performance, there are a few optimizations that could improve it: Preload images If you know you are gonna need all the images, preload them. Without this your slider is gonna appear blank and/...
Emil S. Jørgensen's user avatar
2 votes

Interview project to redesign a website for a property

What immediately negatively stands out to me is the pure JavaScript navigation. The way you have implemented it completely breaks the usability: Forward und back no longer word, nor will page reload ...
RoToRa's user avatar
  • 11k
2 votes
Accepted

Simple application for the game "Mad Lib"

Feedback The code does cache DOM references (which I know we have mentioned in your other posts) using const. All lines appear to have a semi-colon to end the ...
Sᴀᴍ Onᴇᴌᴀ's user avatar
2 votes

Website for a historical society

Bootstrap uses HTML5. So, your doctype should be : <!DOCTYPE html> Basic Template clearly states that the 3 ...
Kiran's user avatar
  • 121
2 votes
Accepted

Simple landing page using Bootstrap

Good start. I would like to add a few minor things: Character encoding As a good practice, the character encoding should be the first element of the head section -...
insertusernamehere's user avatar
2 votes

Replace string of class of button by class so that only one button is filled in using bootstrap buttons

Here's how I refactored your code: ...
Katie.Sun's user avatar
  • 236
2 votes

Master Doughnut Shop Website

First off, I'm not a big fan of Bootstrap. It bloats the HTML with additional elements and classes. Also as a beginner it stops you from learning standard CSS. Avoid using IDs to apply styles. IDs ...
RoToRa's user avatar
  • 11k
1 vote

Semantic markup for card blocks

The one observation I have, is that the two outer elements should be swapped: The outer section has no semantic use, it just contains layout classes. On the other ...
RoToRa's user avatar
  • 11k
1 vote

Bootstrap pagination

Could you give more information on what the point of this is? All it does is change the class of the links. What about the actual pagination itself, which i assume would use AJAX? This code should ...
RoToRa's user avatar
  • 11k
1 vote

Create bootstrap toolbar

Your HTML code is not properly indented and it makes your HTML hard to read, quickly. this is how it should look, (I am surprised that none of your tags were missing their end tags) ...
Malachi's user avatar
  • 28.6k
1 vote
Accepted

Bootstrap responsive layout

This is a small piece of code but few things I'm going to highlight here When using the col class, it's best practice to have this class under a ...
Tolani's user avatar
  • 2,465
1 vote

HTML/bootstrap demo page

Feedback The diverse colors and UI controls look good- nice work! And good work chaining the animation calls for the slide-up/down text (i.e. #text3). Suggestions ...
Sᴀᴍ Onᴇᴌᴀ's user avatar
1 vote
Accepted

Build a quiz game with ReactJS

Here's a couple of things you could simplify: Most of your components have no state, so they can be functional components, which are shorter and cleaner: ...
BenC's user avatar
  • 2,738
1 vote

Interview project to redesign a website for a property

A short review; mcloseNav should be mCloseNav if you follow lowerCamelCase. It should probably even be ...
konijn's user avatar
  • 33.2k

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