From 7384fead9c86c720343121e9d1477fbb268128d9 Mon Sep 17 00:00:00 2001
From: janelle-bouchard <jbouchard@caktusgroup.com>
Date: Tue, 11 Jul 2023 19:38:10 +0000
Subject: [PATCH 1/4] Updated the label on the second new Traffic Stops by Stop
 Purpose and Race Count graph, added description tooltips below the Stop
 Purpose dropdown selector and updated the text for vehicle regulatory
 violation description.

---
 .../Charts/TrafficStops/TrafficStops.js       | 15 +++++++++++++-
 frontend/src/util/displayDefinition.js        | 20 +++++++++++++++++++
 2 files changed, 34 insertions(+), 1 deletion(-)

diff --git a/frontend/src/Components/Charts/TrafficStops/TrafficStops.js b/frontend/src/Components/Charts/TrafficStops/TrafficStops.js
index a7af2b0f..80434544 100644
--- a/frontend/src/Components/Charts/TrafficStops/TrafficStops.js
+++ b/frontend/src/Components/Charts/TrafficStops/TrafficStops.js
@@ -47,6 +47,7 @@ import range from 'lodash.range';
 import LineChart from '../../NewCharts/LineChart';
 import axios from '../../../Services/Axios';
 import NewModal from '../../NewCharts/NewModal';
+import displayDefinition from 'util/displayDefinition';
 
 function TrafficStops(props) {
   const { agencyId } = props;
@@ -493,6 +494,7 @@ function TrafficStops(props) {
               onChange={handleStopPurposeSelect}
               options={[PURPOSE_DEFAULT].concat(stopTypes)}
             />
+            <p style={{ marginTop: '10px' }}>{displayDefinition(purpose)}</p>
             <MonthRangePicker
               agencyId={agencyId}
               dataSet={purpose !== PURPOSE_DEFAULT ? STOPS_BY_REASON : STOPS}
@@ -585,7 +587,7 @@ function TrafficStops(props) {
           <GroupedStopsContainer>
             <LineChart
               data={stopsGroupedByPurposeData.regulatory}
-              title="Regulatory Equipment"
+              title="Regulatory/Equipment"
               maintainAspectRatio={false}
               displayLegend={false}
               yAxisMax={stopsGroupedByPurposeData.max_step_size}
@@ -684,6 +686,17 @@ const BY_REASON_TABLE_COLUMNS = [
   },
 ];
 
+// SPEED_LIMIT_VIOLATION = 1  # Safety Violation
+// STOP_LIGHT_SIGN_VIOLATION = 2  # Safety Violation
+// DRIVING_WHILE_IMPAIRED = 3  # Safety Violation
+// SAFE_MOVEMENT_VIOLATION = 4  # Safety Violation
+// VEHICLE_EQUIPMENT_VIOLATION = 5  # Regulatory and Equipment
+// VEHICLE_REGULATORY_VIOLATION = 6  # Regulatory and Equipment
+// OTHER_MOTOR_VEHICLE_VIOLATION = 9  # Regulatory and Equipment
+// SEAT_BELT_VIOLATION = 7  # Investigatory
+// INVESTIGATION = 8  # Investigatory
+// CHECKPOINT = 10  # Investigatory 
+
 const STOP_PURPOSE_TABLE_COLUMNS = [
   {
     Header: 'Year',
diff --git a/frontend/src/util/displayDefinition.js b/frontend/src/util/displayDefinition.js
index 680707e3..6db2c909 100644
--- a/frontend/src/util/displayDefinition.js
+++ b/frontend/src/util/displayDefinition.js
@@ -11,5 +11,25 @@ export default function displayDefinition(st) {
       'Once a person has been arrested, a law enforcement officer is allowed to search the individual for further contraband and evidence which might add to their criminal charges.',
     'Protective Frisk':
       'A protective frisk can be conducted by a law enforcement officer in efforts to find any weapons which could be used against the officer. This can only be done when the officer has reason to believe the stopped individual may be dangerous and holding weapons.',
+    Checkpoint:
+      'A checkpoint can be a barrier, or a manned entrance, where drivers are subjected to a security check, often by law enforcement. For example, a checkpoint could be where police set up roadblocks so they can stop and inspect all (or almost all) drivers and vehicles that pass through.',
+    'Other Motor Vehicle Violation':
+      'If the stop falls within this category, the officer may have stopped an individual for some other violation not listed within the above/below categories',
+    Investigation:
+      'An investigation is a systematic, thorough attempt to learn the facts about a potential case/crime. An investigation is the process in which law enforcement may go about gathering more information about the case in question. For example, a car may be stopped during an investigation if the police are looking for a suspect to a crime, and the car matches the description of the suspect in question.',
+    'Seat Belt Violation':
+      'Under NC law, each occupant of a vehicle is required to be fastened by a seat belt when it is in motion. A seatbelt violation is when one or more occupant of a vehicle is not properly wearing a seatbelt while the vehicle is in motion.',
+    'Vehicle Regulatory Violation':
+      'A vehicle regulatory violation is a violation of regulatory laws regarding motor vehicles. For example, this could look like having expired license plates, an expired tag, or driving without a license.',
+    'Vehicle Equipment Violation':
+      'A vehicle equipment violation is considered a non-moving violation, and often refers to faulty equipment in a vehicle. Improper equipment may describe a vehicle having equipment or mechanical problems such as dangerous tires, improper muffler, or a broken speedometer. Some other common vehicle equipment violations people might be stopped for are broken/inoperable lights, tinted windows that are too dark, cracked windshield wipers.',
+    'Safe Movement Violation':
+      'A safe movement violation in North Carolina is a violation of safe movement practices while driving. This can apply to many unsafe driving practices, such as unsafe lane changes or improper turns.',
+    'Driving While Impaired':
+      'If someone is stopped for suspicion of driving while impaired, the officer may claim to have reason to believe that the person is operating the car while under the influence of any impairing substance, such as alcohol or drugs.',
+    'Stop Light/Sign Violation':
+      'Someone may be stopped for this kind of violation, for example, by failing to stop completely at a stop sign, by running a red light, or by disregarding a particular road sign.',
+    'Speed Limit Violation':
+      'A speed limit violation is when an individual is stopped for driving faster than the posted speed limit in a given area, and on a given road. Someone may be stopped for this kind of violation if an officer suspects them of driving over the posted speed limit, or too fast for the current road conditions.',
   }[st];
 }

From 21965080aba558138a15f7b1b243196e12fcd07f Mon Sep 17 00:00:00 2001
From: janelle-bouchard <jbouchard@caktusgroup.com>
Date: Tue, 18 Jul 2023 14:29:09 +0000
Subject: [PATCH 2/4] Fixed the failing tests.

---
 .../Components/Charts/TrafficStops/TrafficStops.js  | 13 +------------
 1 file changed, 1 insertion(+), 12 deletions(-)

diff --git a/frontend/src/Components/Charts/TrafficStops/TrafficStops.js b/frontend/src/Components/Charts/TrafficStops/TrafficStops.js
index 80434544..02707f60 100644
--- a/frontend/src/Components/Charts/TrafficStops/TrafficStops.js
+++ b/frontend/src/Components/Charts/TrafficStops/TrafficStops.js
@@ -47,7 +47,7 @@ import range from 'lodash.range';
 import LineChart from '../../NewCharts/LineChart';
 import axios from '../../../Services/Axios';
 import NewModal from '../../NewCharts/NewModal';
-import displayDefinition from 'util/displayDefinition';
+import displayDefinition from '../../../util/displayDefinition';
 
 function TrafficStops(props) {
   const { agencyId } = props;
@@ -686,17 +686,6 @@ const BY_REASON_TABLE_COLUMNS = [
   },
 ];
 
-// SPEED_LIMIT_VIOLATION = 1  # Safety Violation
-// STOP_LIGHT_SIGN_VIOLATION = 2  # Safety Violation
-// DRIVING_WHILE_IMPAIRED = 3  # Safety Violation
-// SAFE_MOVEMENT_VIOLATION = 4  # Safety Violation
-// VEHICLE_EQUIPMENT_VIOLATION = 5  # Regulatory and Equipment
-// VEHICLE_REGULATORY_VIOLATION = 6  # Regulatory and Equipment
-// OTHER_MOTOR_VEHICLE_VIOLATION = 9  # Regulatory and Equipment
-// SEAT_BELT_VIOLATION = 7  # Investigatory
-// INVESTIGATION = 8  # Investigatory
-// CHECKPOINT = 10  # Investigatory 
-
 const STOP_PURPOSE_TABLE_COLUMNS = [
   {
     Header: 'Year',

From b2234e72ef7a6abc715efd2c64eaf9cdcdd0a360 Mon Sep 17 00:00:00 2001
From: janelle-bouchard <jbouchard@caktusgroup.com>
Date: Wed, 19 Jul 2023 18:53:40 +0000
Subject: [PATCH 3/4] Removed the 10px margin when the stop purpose is the
 default.

---
 frontend/src/Components/Charts/TrafficStops/TrafficStops.js | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/frontend/src/Components/Charts/TrafficStops/TrafficStops.js b/frontend/src/Components/Charts/TrafficStops/TrafficStops.js
index 02707f60..8e32da4b 100644
--- a/frontend/src/Components/Charts/TrafficStops/TrafficStops.js
+++ b/frontend/src/Components/Charts/TrafficStops/TrafficStops.js
@@ -494,7 +494,11 @@ function TrafficStops(props) {
               onChange={handleStopPurposeSelect}
               options={[PURPOSE_DEFAULT].concat(stopTypes)}
             />
-            <p style={{ marginTop: '10px' }}>{displayDefinition(purpose)}</p>
+
+            {purpose !== PURPOSE_DEFAULT ? (
+              <p style={{ marginTop: '10px' }}>{displayDefinition(purpose)}</p>
+            ) : null}
+
             <MonthRangePicker
               agencyId={agencyId}
               dataSet={purpose !== PURPOSE_DEFAULT ? STOPS_BY_REASON : STOPS}

From 11b82585dec1f082a5b41f90e1c2b7a28a209527 Mon Sep 17 00:00:00 2001
From: Aristotel Fani <afani1997@gmail.com>
Date: Fri, 21 Jul 2023 17:53:39 -0400
Subject: [PATCH 4/4] Use double and to show stop purpose definition

---
 frontend/src/Components/Charts/TrafficStops/TrafficStops.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/frontend/src/Components/Charts/TrafficStops/TrafficStops.js b/frontend/src/Components/Charts/TrafficStops/TrafficStops.js
index 8e32da4b..5af3ef36 100644
--- a/frontend/src/Components/Charts/TrafficStops/TrafficStops.js
+++ b/frontend/src/Components/Charts/TrafficStops/TrafficStops.js
@@ -495,9 +495,9 @@ function TrafficStops(props) {
               options={[PURPOSE_DEFAULT].concat(stopTypes)}
             />
 
-            {purpose !== PURPOSE_DEFAULT ? (
+            {purpose !== PURPOSE_DEFAULT && (
               <p style={{ marginTop: '10px' }}>{displayDefinition(purpose)}</p>
-            ) : null}
+            )}
 
             <MonthRangePicker
               agencyId={agencyId}