fixed review

app/assets/stylesheets/custom.scss

@@ -75,96 +75,3 @@ footer {
     }
   }
 }
-
-// Top page 
-// search 
-
-
-// latest-job
-
-.latest-job {
-  .job-item {
-    font-size: 14px;
-    .job-title {
-      text-decoration: none;
-      color: black;
-      font-size: 20px;
-      font-weight: 500;
-    }
-    .job-caption {
-      line-height: 18px;
-      .job-company {
-        text-decoration: none;
-        font-size: 14px;
-        color: $gray-700;
-      }
-      .job-salary {
-        color: #008563;
-        margin: 0;
-      }
-      .job-locations {
-        ul {
-          list-style: none;
-          margin: 0 0 6px 0;
-          padding: 0;
-          color: $gray-700;
-        }
-      }
-      .job-desc {
-        overflow: hidden;
-        text-overflow: ellipsis;
-        line-height: 18px;
-        -webkit-line-clamp: 2;
-        max-height: 36px;
-        display: -webkit-box;
-        -webkit-box-orient: vertical;
-      }
-    }
-  }
-}
-
-// top_cities
-
-.top_cities {
-  .city-item {
-    margin-bottom: 12px;
-    a {
-      font-size: 18px;
-      text-decoration: none;
-      color: #287ab9;
-      margin: 10px 0;
-    }
-  }
-  .all-cities-btn {
-    color: #287ab9;
-    font-size: 20px;
-    text-decoration: none;
-  }
-  .all-cities-btn:hover {
-    text-decoration: underline;
-  }
-}
-
-// top_industries
-
-.top_industries {
-  .industry-item {
-    margin-bottom: 12px;
-    a {
-      font-size: 18px;
-      text-decoration: none;
-      color: #287ab9;
-      margin: 10px 0;
-    }
-  }
-  .all-industries-btn {
-    color: #287ab9;
-    font-size: 20px;
-    text-decoration: none;
-  }
-  .all-industries-btn:hover {
-    text-decoration: underline;
-  }
-}
-
-// cities list 
\ No newline at end of file

app/assets/stylesheets/top.scss

+$white:    #fff;
+$gray-100: #f8f9fa;
+$gray-200: #e9ecef;
+$gray-300: #dee2e6;
+$gray-400: #ced4da;
+$gray-500: #adb5bd;
+$gray-600: #6c757d;
+$gray-700: #495057;
+$gray-800: #343a40;
+$gray-900: #212529;
+$black:    #000;
+// Top page 
+// search 
+
+
+// latest-job
+
+.latest-job {
+  .job-item {
+    font-size: 14px;
+    .job-title {
+      text-decoration: none;
+      color: black;
+      font-size: 20px;
+      font-weight: 500;
+    }
+    .job-caption {
+      line-height: 18px;
+      .job-company {
+        text-decoration: none;
+        font-size: 14px;
+        color: $gray-700;
+      }
+      .job-salary {
+        color: #008563;
+        margin: 0;
+      }
+      .job-locations {
+        ul {
+          list-style: none;
+          margin: 0 0 6px 0;
+          padding: 0;
+          color: $gray-700;
+        }
+      }
+      .job-desc {
+        overflow: hidden;
+        text-overflow: ellipsis;
+        line-height: 18px;
+        -webkit-line-clamp: 2;
+        max-height: 36px;
+        display: -webkit-box;
+        -webkit-box-orient: vertical;
+      }
+    }
+  }
+}
+
+// top_cities
+
+.top_cities {
+  .city-item {
+    margin-bottom: 12px;
+    a {
+      font-size: 18px;
+      text-decoration: none;
+      color: #287ab9;
+      margin: 10px 0;
+    }
+  }
+  .all-cities-btn {
+    color: #287ab9;
+    font-size: 20px;
+    text-decoration: none;
+  }
+  .all-cities-btn:hover {
+    text-decoration: underline;
+  }
+}
+
+// top_industries
+
+.top_industries {
+  .industry-item {
+    margin-bottom: 12px;
+    a {
+      font-size: 18px;
+      text-decoration: none;
+      color: #287ab9;
+      margin: 10px 0;
+    }
+  }
+  .all-industries-btn {
+    color: #287ab9;
+    font-size: 20px;
+    text-decoration: none;
+  }
+  .all-industries-btn:hover {
+    text-decoration: underline;
+  }
+}

app/models/industry.rb

 class Industry < ApplicationRecord
-  scope :jobs_count, -> { joins(:jobs).group(:name).count.sort_by(&:first) }
-
   TOP_JOB_COUNT = 9
 
   has_and_belongs_to_many :jobs

app/views/jobs/index.html.slim

@@ -15,7 +15,7 @@
   - @jobs.each do |job|
     / job
     .job-item
-      = link_to job.title, job_path(Job.find_by(title: job.title).id), class: 'job-title fs-3 text-decoration-none text-reset'
+      = link_to job.title, job, class: 'job-title fs-3 text-decoration-none text-reset'
       .job-caption
         = link_to job.company.name, '#', class: 'job-company text-decoration-none text-secondary'
         p.job-salary.text-success

app/views/jobs/show.html.slim

+ruby: 
+  city_list = @job.cities
+
+/ html
 .container.my-5
   / breadcrumb 
   .breadcrumb
@@ -5,7 +9,6 @@
       'Home
     span.d-block.mx-2
       |  >  
-    - city_list = @job.cities
     - city_list.each_with_index do |city, i|
       = link_to job_list_path(model: :city, slug: city.slug), class: 'text-decoration-none text-info' do
         = city.name

app/views/top/index.html.slim

@@ -9,7 +9,7 @@
     hr.my-2
     - @latest_jobs.each do |job|
       .job-item.mb-4
-        = link_to job.title, job_path(job.id), class: 'job-title'
+        = link_to job.title, job, class: 'job-title'
         .job-caption
           = link_to job.company.name, '#', class: 'job-company'
           p.job-salary