Tag Info

Accepted

Country Guessing Game

The screen design looks very nice. It is minimalistic, yet pleasing. It almost perfectly fits on the small screen of my phone (only sometimes, I have to scroll down to see all the answer buttons). The ...
• 21.1k

Simple checkboxes with React

I think everything Sam Onela said is great. I'm gonna add a few more things. Don't modify your state directly In general, it's considered bad practice to directly modify your state object. According ...

Country Guessing Game

Feedback Nice use of arrow functions, the fetch API (and associated promises), etc. The layout looks nice as well. Suggestions/Review Points Options could appear multiple times The code in ...
• 26.4k
Accepted

batch progress visualization In React.js

The function render() in only supposed to render. It is not supposed to compute some style, set variables, have any logic other than render: (Functions Names Should ...
• 188
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. ...
• 13.2k
Accepted

TODO in React w/ Redux

I'm not exactly sure of your environment (I'm going to assume you're using Node), and with this assumption, I want to address a few things starting with specific things, then more general: Specific ...
• 261
Accepted

Jest / Enzyme test for a React component to calculate average data speed

There are two distinct parts in your test: Testing the rendering of AvgSpeed component. Testing the logic of getAverage() and <...
• 2,064

batch progress visualization In React.js

The answer by Orlyyn explains it well. Just adding to it: You can probably destructure your variables, this helps with readability and reuse. Since you are not using the state of the class you can ...
• 51
Accepted

I think there are many things you could improve, I will try to review your code and give some pointers. Instead of using React.Component, you should directly ...
• 1,508
Accepted

• 1,952
Accepted

Class to create HTML heading tags in React

Choosing the heading level at runtime: According to the official docs, you cannot use expressions within JSX types. However, you can use capitalized variables: ...
• 1,925

onChange vs onKeyPress for input in React

Don't complicate your code; the first solution is just fine but needs some tweaking: ...
Accepted

Validating Array before rendering

typeof activeEvents === 'object' && activeEvents !== null can be true for both Array ...
• 598
Accepted

Different class name based on condition in React

if you're just trying to decide between number--high number--medium or '' based on the ...
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 ...
• 431
Accepted

React setState function in useEffect

Short answer: You can include setExpanded in the dependency array. Long answer: The function passed to useEffect will fire ...
• 56

How many useState and useEffect instances is conventional: one per page or one per app?

I don't normally code in JavaScript but here's some problems I see with the code. When skimming your code it's hard to tell where a lot of function's argument and body, end and begin. this is because ...
• 42.2k

React Native screen to add a city to a weather app using a REST API

The first mistake: API key is public maybe you could store it in react-native-keychain. Secondly, you should create three files for style(component), for view(component), for function(API&(...
Accepted

React calculator

Fully functional example : ...
• 340
Accepted

Mapping JSON object to HTML

A few things: I'm not sure why you need to map over the top level object? Are there top-level keys other than dataset that you're interested in? From your example,...
• 146
Accepted

Static single page app with routing

I haven't quite understood what Babel is used for at all. Babel is used for transpilation - the process of converting unsupported/future syntax into one that is supported at the moment. For instance, ...
• 25k

React Native - Combining login and signup pages into one component

React is component based, so i think you should not combine it, you should split component for the footer, component for the header, component for the container and if you want to render inside the ...
• 137

Unit tests for React component to submit an input form with validation

One thing that could help a lot is to realize that tests verify behavior, not implementation. This means you shouldn't be validating these things: should have a ...
• 1,100
Accepted

Simple checkboxes with React

Main Question Can somebody please tell me what would have made my code more "correct"? It is difficult to know exactly what would have made my code more "correct" but it does look like the method ...
• 26.4k

React container component

You don't have to put content as a prop, you can do this: ...
Accepted

How to set state of multiple properties in one event handler (React)

I find it fine as it is, but if you really want to keep a single method, you can parametrize it by the name of the key to set: ...

React For loop component

This approach is not bad but I'd highlight a few issues: You wrap each element with a <div>, it's unnecessary and for a very long list you create many ...
• 10.4k
Accepted

TypeScript implicit 'any' type with computed property key

You very rarely need to use any in TypeScript because its type-system is so expressive. The problem is that copyFormat is ...
• 334

Minimize Hard Coding in React Native

To clean the main render, we can extract the below JSX into a method and render that method into the main render. ...
• 51