rails_pulse 0.2.2 → 0.2.4

data/app/models/rails_pulse/queries/charts/average_query_times.rb

@@ -2,18 +2,21 @@ module RailsPulse
   module Queries
     module Charts
       class AverageQueryTimes
-        def initialize(ransack_query:, period_type: nil, query: nil, start_time: nil, end_time: nil, start_duration: nil)
+        def initialize(ransack_query:, period_type: nil, query: nil, start_time: nil, end_time: nil, start_duration: nil, disabled_tags: [], show_non_tagged: true)
           @ransack_query = ransack_query
           @period_type = period_type
           @query = query
           @start_time = start_time
           @end_time = end_time
           @start_duration = start_duration
+          @disabled_tags = disabled_tags
+          @show_non_tagged = show_non_tagged
         end
 
-        def to_rails_chart
-          # The ransack query already contains the correct filters, just add period_type
+        def to_chart_data
+          # The ransack query already contains the correct filters, just add period_type and tag filters
           summaries = @ransack_query.result(distinct: false)
+            .with_tag_filters(@disabled_tags, @show_non_tagged)
             .where(period_type: @period_type)
             .group(:period_start)
             .having("AVG(avg_duration) > ?", @start_duration || 0)

data/app/models/rails_pulse/queries/tables/index.rb

@@ -2,13 +2,14 @@ module RailsPulse
   module Queries
     module Tables
       class Index
-        def initialize(ransack_query:, period_type: nil, start_time:, params:, query: nil, disabled_tags: [])
+        def initialize(ransack_query:, period_type: nil, start_time:, params:, query: nil, disabled_tags: [], show_non_tagged: true)
           @ransack_query = ransack_query
           @period_type = period_type
           @start_time = start_time
           @params = params
           @query = query
           @disabled_tags = disabled_tags
+          @show_non_tagged = show_non_tagged
         end
 
         def to_table
@@ -23,10 +24,19 @@ module RailsPulse
             )
 
           # Apply tag filters by excluding queries with disabled tags
-          @disabled_tags.each do |tag|
+          # Separate "non_tagged" from actual tags (it's a virtual tag)
+          actual_disabled_tags = @disabled_tags.reject { |tag| tag == "non_tagged" }
+
+          # Exclude queries with actual disabled tags
+          actual_disabled_tags.each do |tag|
             base_query = base_query.where.not("rails_pulse_queries.tags LIKE ?", "%#{tag}%")
           end
 
+          # Exclude non-tagged queries if show_non_tagged is false
+          unless @show_non_tagged
+            base_query = base_query.where("rails_pulse_queries.tags IS NOT NULL AND rails_pulse_queries.tags != '[]'")
+          end
+
           base_query = base_query.where(summarizable_id: @query.id) if @query
 
           # Apply grouping and aggregation

data/app/models/rails_pulse/requests/charts/average_response_times.rb

@@ -2,21 +2,27 @@ module RailsPulse
   module Requests
     module Charts
       class AverageResponseTimes
-        def initialize(ransack_query:, period_type: nil, route: nil, start_time: nil, end_time: nil, start_duration: nil)
+        def initialize(ransack_query:, period_type: nil, route: nil, start_time: nil, end_time: nil, start_duration: nil, disabled_tags: [], show_non_tagged: true)
           @ransack_query = ransack_query
           @period_type = period_type
           @route = route
           @start_time = start_time
           @end_time = end_time
           @start_duration = start_duration
+          @disabled_tags = disabled_tags
+          @show_non_tagged = show_non_tagged
         end
 
-        def to_rails_chart
-          summaries = @ransack_query.result(distinct: false).where(
-            summarizable_type: "RailsPulse::Request",
-            summarizable_id: 0,  # Overall request summaries
-            period_type: @period_type
-          )
+        def to_chart_data
+          # Note: Overall request summaries (summarizable_id: 0) are not filtered by tags
+          # as they aggregate all requests regardless of route tags
+          summaries = @ransack_query.result(distinct: false)
+            .with_tag_filters(@disabled_tags, @show_non_tagged)
+            .where(
+              summarizable_type: "RailsPulse::Request",
+              summarizable_id: 0,  # Overall request summaries
+              period_type: @period_type
+            )
 
           summaries = summaries
             .group(:period_start)

data/app/models/rails_pulse/routes/cards/average_response_times.rb

@@ -2,8 +2,10 @@ module RailsPulse
   module Routes
     module Cards
       class AverageResponseTimes
-        def initialize(route:)
+        def initialize(route:, disabled_tags: [], show_non_tagged: true)
           @route = route
+          @disabled_tags = disabled_tags
+          @show_non_tagged = show_non_tagged
         end
 
         def to_metric_card
@@ -11,11 +13,13 @@ module RailsPulse
           previous_7_days = 14.days.ago.beginning_of_day
 
           # Single query to get all aggregated metrics with conditional sums
-          base_query = RailsPulse::Summary.where(
-            summarizable_type: "RailsPulse::Route",
-            period_type: "day",
-            period_start: 2.weeks.ago.beginning_of_day..Time.current
-          )
+          base_query = RailsPulse::Summary
+            .with_tag_filters(@disabled_tags, @show_non_tagged)
+            .where(
+              summarizable_type: "RailsPulse::Route",
+              period_type: "day",
+              period_start: 2.weeks.ago.beginning_of_day..Time.current
+            )
           base_query = base_query.where(summarizable_id: @route.id) if @route
 
           metrics = base_query.select(

data/app/models/rails_pulse/routes/cards/error_rate_per_route.rb

@@ -2,8 +2,10 @@ module RailsPulse
   module Routes
     module Cards
       class ErrorRatePerRoute
-        def initialize(route: nil)
+        def initialize(route: nil, disabled_tags: [], show_non_tagged: true)
           @route = route
+          @disabled_tags = disabled_tags
+          @show_non_tagged = show_non_tagged
         end
 
         def to_metric_card
@@ -11,11 +13,13 @@ module RailsPulse
           previous_7_days = 14.days.ago.beginning_of_day
 
           # Single query to get all error metrics with conditional aggregation
-          base_query = RailsPulse::Summary.where(
-            summarizable_type: "RailsPulse::Route",
-            period_type: "day",
-            period_start: 2.weeks.ago.beginning_of_day..Time.current
-          )
+          base_query = RailsPulse::Summary
+            .with_tag_filters(@disabled_tags, @show_non_tagged)
+            .where(
+              summarizable_type: "RailsPulse::Route",
+              period_type: "day",
+              period_start: 2.weeks.ago.beginning_of_day..Time.current
+            )
           base_query = base_query.where(summarizable_id: @route.id) if @route
 
           metrics = base_query.select(

data/app/models/rails_pulse/routes/cards/percentile_response_times.rb

@@ -2,8 +2,10 @@ module RailsPulse
   module Routes
     module Cards
       class PercentileResponseTimes
-        def initialize(route: nil)
+        def initialize(route: nil, disabled_tags: [], show_non_tagged: true)
           @route = route
+          @disabled_tags = disabled_tags
+          @show_non_tagged = show_non_tagged
         end
 
         def to_metric_card
@@ -11,11 +13,13 @@ module RailsPulse
           previous_7_days = 14.days.ago.beginning_of_day
 
           # Single query to get all P95 metrics with conditional aggregation
-          base_query = RailsPulse::Summary.where(
-            summarizable_type: "RailsPulse::Route",
-            period_type: "day",
-            period_start: 2.weeks.ago.beginning_of_day..Time.current
-          )
+          base_query = RailsPulse::Summary
+            .with_tag_filters(@disabled_tags, @show_non_tagged)
+            .where(
+              summarizable_type: "RailsPulse::Route",
+              period_type: "day",
+              period_start: 2.weeks.ago.beginning_of_day..Time.current
+            )
           base_query = base_query.where(summarizable_id: @route.id) if @route
 
           metrics = base_query.select(

data/app/models/rails_pulse/routes/cards/request_count_totals.rb

@@ -2,8 +2,10 @@ module RailsPulse
   module Routes
     module Cards
       class RequestCountTotals
-        def initialize(route: nil)
+        def initialize(route: nil, disabled_tags: [], show_non_tagged: true)
           @route = route
+          @disabled_tags = disabled_tags
+          @show_non_tagged = show_non_tagged
         end
 
         def to_metric_card
@@ -11,11 +13,13 @@ module RailsPulse
           previous_7_days = 14.days.ago.beginning_of_day
 
           # Single query to get all count metrics with conditional aggregation
-          base_query = RailsPulse::Summary.where(
-            summarizable_type: "RailsPulse::Route",
-            period_type: "day",
-            period_start: 2.weeks.ago.beginning_of_day..Time.current
-          )
+          base_query = RailsPulse::Summary
+            .with_tag_filters(@disabled_tags, @show_non_tagged)
+            .where(
+              summarizable_type: "RailsPulse::Route",
+              period_type: "day",
+              period_start: 2.weeks.ago.beginning_of_day..Time.current
+            )
           base_query = base_query.where(summarizable_id: @route.id) if @route
 
           metrics = base_query.select(

data/app/models/rails_pulse/routes/charts/average_response_times.rb

@@ -2,20 +2,24 @@ module RailsPulse
   module Routes
     module Charts
       class AverageResponseTimes
-        def initialize(ransack_query:, period_type: nil, route: nil, start_time: nil, end_time: nil, start_duration: nil)
+        def initialize(ransack_query:, period_type: nil, route: nil, start_time: nil, end_time: nil, start_duration: nil, disabled_tags: [], show_non_tagged: true)
           @ransack_query = ransack_query
           @period_type = period_type
           @route = route
           @start_time = start_time
           @end_time = end_time
           @start_duration = start_duration
+          @disabled_tags = disabled_tags
+          @show_non_tagged = show_non_tagged
         end
 
-        def to_rails_chart
-          summaries = @ransack_query.result(distinct: false).where(
-            summarizable_type: "RailsPulse::Route",
-            period_type: @period_type
-          )
+        def to_chart_data
+          summaries = @ransack_query.result(distinct: false)
+            .with_tag_filters(@disabled_tags, @show_non_tagged)
+            .where(
+              summarizable_type: "RailsPulse::Route",
+              period_type: @period_type
+            )
 
           summaries = summaries.where(summarizable_id: @route.id) if @route
           summaries = summaries

data/app/models/rails_pulse/routes/tables/index.rb

@@ -2,12 +2,13 @@ module RailsPulse
   module Routes
     module Tables
       class Index
-        def initialize(ransack_query:, period_type: nil, start_time:, params:, disabled_tags: [])
+        def initialize(ransack_query:, period_type: nil, start_time:, params:, disabled_tags: [], show_non_tagged: true)
           @ransack_query = ransack_query
           @period_type = period_type
           @start_time = start_time
           @params = params
           @disabled_tags = disabled_tags
+          @show_non_tagged = show_non_tagged
         end
 
         def to_table
@@ -24,10 +25,19 @@ module RailsPulse
             )
 
           # Apply tag filters by excluding routes with disabled tags
-          @disabled_tags.each do |tag|
+          # Separate "non_tagged" from actual tags (it's a virtual tag)
+          actual_disabled_tags = @disabled_tags.reject { |tag| tag == "non_tagged" }
+
+          # Exclude routes with actual disabled tags
+          actual_disabled_tags.each do |tag|
             base_query = base_query.where.not("rails_pulse_routes.tags LIKE ?", "%#{tag}%")
           end
 
+          # Exclude non-tagged routes if show_non_tagged is false
+          unless @show_non_tagged
+            base_query = base_query.where("rails_pulse_routes.tags IS NOT NULL AND rails_pulse_routes.tags != '[]'")
+          end
+
           base_query = base_query.where(summarizable_id: @route.id) if @route
 
           # Apply grouping and aggregation

data/app/models/rails_pulse/summary.rb

@@ -33,6 +33,61 @@ module RailsPulse
       where(summarizable_type: "RailsPulse::Request", summarizable_id: 0)
     }
 
+    # Tag filtering scope for charts and metrics
+    # Filters summaries based on disabled tags in the underlying route/query
+    scope :with_tag_filters, ->(disabled_tags = [], show_non_tagged = true) {
+      # Separate "non_tagged" from actual tags (it's a virtual tag)
+      actual_disabled_tags = disabled_tags.reject { |tag| tag == "non_tagged" }
+
+      # Return early if no filters are applied
+      return all if actual_disabled_tags.empty? && show_non_tagged
+
+      # Determine which table to join based on summarizable_type
+      # We need to handle both Route and Query summaries
+      relation = all
+
+      # Filter route summaries
+      route_ids = RailsPulse::Route.all
+
+      # Exclude routes with disabled tags
+      actual_disabled_tags.each do |tag|
+        route_ids = route_ids.where.not("tags LIKE ?", "%#{tag}%")
+      end
+
+      # Exclude non-tagged routes if show_non_tagged is false
+      route_ids = route_ids.where("tags IS NOT NULL AND tags != '[]'") unless show_non_tagged
+
+      route_ids = route_ids.pluck(:id)
+
+      # Filter query summaries
+      query_ids = RailsPulse::Query.all
+
+      # Exclude queries with disabled tags
+      actual_disabled_tags.each do |tag|
+        query_ids = query_ids.where.not("tags LIKE ?", "%#{tag}%")
+      end
+
+      # Exclude non-tagged queries if show_non_tagged is false
+      query_ids = query_ids.where("tags IS NOT NULL AND tags != '[]'") unless show_non_tagged
+
+      query_ids = query_ids.pluck(:id)
+
+      # Apply filters: include only summaries for filtered routes/queries
+      # If no routes/queries match the filter, we need to ensure nothing is returned
+      # Use -1 as an impossible ID instead of 0 (which might be used for aggregates)
+      relation = relation.where(
+        "(" \
+        "  (summarizable_type = 'RailsPulse::Route' AND summarizable_id IN (?)) OR " \
+        "  (summarizable_type = 'RailsPulse::Query' AND summarizable_id IN (?)) OR " \
+        "  (summarizable_type = 'RailsPulse::Request')" \
+        ")",
+        route_ids.presence || [ -1 ],
+        query_ids.presence || [ -1 ]
+      )
+
+      relation
+    }
+
     # Ransack configuration
     def self.ransackable_attributes(auth_object = nil)
       %w[

data/app/views/layouts/rails_pulse/_global_filters.html.erb

@@ -15,8 +15,15 @@
   <div data-rails-pulse--global-filters-target="wrapper" data-action="click->rails-pulse--global-filters#closeOnClickOutside" style="display: none; position: fixed; inset: 0; background-color: rgba(0, 0, 0, 0.8); z-index: 1000; align-items: center; justify-content: center;">
     <div class="dialog" data-rails-pulse--global-filters-target="dialog" style="position: relative; opacity: 1; transform: scale(1);">
       <div class="dialog__content">
-        <h2 class="text-lg font-semibold mb-4">Global Filters</h2>
-        <p class="text-sm text-subtle mb-4">Set default time filters that persist across all pages. These can be overridden by page-specific filters.</p>
+        <div class="flex items-start justify-between">
+          <div>
+            <h2 class="text-lg font-semibold mbe-4">Global Filters</h2>
+            <p class="text-sm text-subtle">Set default time filters that persist across all pages. These can be overridden by page-specific filters.</p>
+          </div>
+          <%= link_to '#', "aria-label": "Close", data: { action: "rails-pulse--global-filters#close" }, class: "text-subtle hover:text-default" do %>
+            <%= rails_pulse_icon 'x', width: '20' %>
+          <% end %>
+        </div>
 
         <%= form_with url: settings_global_filters_path, method: :patch, local: true, data: { action: "submit->rails-pulse--global-filters#submit" } do |form| %>
           <div class="flex flex-col gap mb-4">

data/app/views/rails_pulse/components/_active_filters.html.erb

@@ -0,0 +1,36 @@
+<% global_filters = session_global_filters %>
+<% disabled_tags = global_filters['disabled_tags'] || [] %>
+<% has_date_filters = global_filters['start_time'].present? && global_filters['end_time'].present? %>
+<% has_performance_filter = global_filters['performance_threshold'].present? %>
+<% has_tag_filters = disabled_tags.any? || session[:show_non_tagged] == false %>
+<% has_any_filters = has_date_filters || has_performance_filter || has_tag_filters %>
+
+<% if has_any_filters %>
+  <div class="flex items-center gap text-sm">
+    Filtered:
+    <% if has_date_filters %>
+      <% start_time = Time.parse(global_filters['start_time']) %>
+      <% end_time = Time.parse(global_filters['end_time']) %>
+      <span class="badge badge--secondary"><%= start_time.strftime("%b %d, %Y %-I:%M %p") %> - <%= end_time.strftime("%b %d, %Y %-I:%M %p") %></span>
+    <% end %>
+
+    <% if has_performance_filter %>
+      <% threshold_label = case global_filters['performance_threshold']
+        when 'slow' then 'Slow and Above'
+        when 'very_slow' then 'Very Slow and Above'
+        when 'critical' then 'Critical'
+        else global_filters['performance_threshold'].titleize
+        end %>
+      <span class="badge badge--secondary"><%= threshold_label %></span>
+    <% end %>
+
+    <% if has_tag_filters %>
+      <% disabled_tags.each do |tag| %>
+        <span class="badge badge--secondary"><%= tag.humanize %></span>
+      <% end %>
+      <% if session[:show_non_tagged] == false %>
+        <span class="badge badge--secondary">Non tagged hidden</span>
+      <% end %>
+    <% end %>
+  </div>
+<% end %>

data/app/views/rails_pulse/components/_metric_card.html.erb

@@ -9,7 +9,7 @@
   trend_amount = data[:trend_amount]
   trend_text = data[:trend_text]
 %>
-<div class="grid-item" data-controller="rails-pulse--chart" id="<%= id %>">
+<div class="grid-item" id="<%= id %>">
   <%= render 'rails_pulse/components/panel', { title: title, card_classes: 'card-compact' } do %>
     <div class="row mbs-2" style="--columns: 2; align-items: center; margin-bottom: 0;">
       <div class="grid-item">
@@ -38,7 +38,7 @@
               }
             )
           %>
-          <%= bar_chart chart_data, height: "100%", options: chart_options %>
+          <%= render_stimulus_chart chart_data, type: 'bar', height: "100%", options: chart_options %>
         </div>
       </div>
     </div>

data/app/views/rails_pulse/components/_page_header.html.erb

@@ -16,5 +16,9 @@
     <div class="breadcrumb-tags">
       <%= render 'rails_pulse/tags/tag_manager', taggable: taggable %>
     </div>
+  <% elsif defined?(show_active_filters) && show_active_filters %>
+    <div class="breadcrumb-tags">
+      <%= render 'rails_pulse/components/active_filters' %>
+    </div>
   <% end %>
 </div>

data/app/views/rails_pulse/components/_sparkline_stats.html.erb

@@ -2,7 +2,7 @@
   <h4 class="text-xl mbs-1 font-bold"><%= summary %></h4>
 </div>
 <div class="chart-container chart-container--slim">
-  <%= bar_chart chart_data, height: "100%", options: sparkline_chart_options %>
+  <%= render_stimulus_chart chart_data, type: 'bar', height: "100%", options: sparkline_chart_options %>
 </div>
 <div>
   <span class="badge badge--<%= trend_direction == "down" ? "positive" : "negative" %>-inverse p-0">

data/app/views/rails_pulse/components/_table_pagination.html.erb

@@ -7,7 +7,7 @@
          data-rails-pulse--pagination-url-value="<%= rails_pulse.pagination_limit_path %>">
       <label for="pagination_limit" class="text-sm font-medium">Rows per page</label>
       <%= select_tag :limit,
-        options_for_select([[10, 10], [20, 20], [30, 30], [40, 40], [50, 50]], @pagy.vars[:items]),
+        options_for_select([[10, 10], [20, 20], [30, 30], [40, 40], [50, 50]], pagy_items(@pagy)),
         {
           id: "pagination_limit",
           class: "input",
@@ -21,22 +21,24 @@
       %>
     </div>
 
-    <div class="text-sm font-medium"><%= "Page #{@pagy.page} of #{@pagy.pages}" %></div>
+    <div class="text-sm font-medium"><%= "Page #{@pagy.page} of #{@pagy.last}" %></div>
 
     <nav class="flex items-center gap shrink-0" style="--btn-padding: .5rem;" aria-label="Pagination">
-      <%= link_to pagy_url_for(@pagy, 1), class: "btn", aria: { disabled: @pagy.prev.nil? }.compact_blank do %>
+      <% previous_page = pagy_previous(@pagy) %>
+      <% next_page = pagy_next(@pagy) %>
+      <%= link_to pagy_page_url(@pagy, 1), class: "btn", aria: { disabled: previous_page.nil? }.compact_blank do %>
         <%= rails_pulse_icon 'chevrons-left', width: '16', height: '16' %>
         <span class="sr-only">Go to first page</span>
       <% end %>
-      <%= link_to pagy_url_for(@pagy, @pagy.prev || @pagy.page), class: "btn", aria: { disabled: @pagy.prev.nil? }.compact_blank do %>
+      <%= link_to pagy_page_url(@pagy, previous_page || @pagy.page), class: "btn", aria: { disabled: previous_page.nil? }.compact_blank do %>
         <%= rails_pulse_icon 'chevron-left', width: '16', height: '16' %>
         <span class="sr-only">Go to previous page</span>
       <% end %>
-      <%= link_to pagy_url_for(@pagy, @pagy.next || @pagy.page), class: "btn", aria: { disabled: @pagy.next.nil? }.compact_blank do %>
+      <%= link_to pagy_page_url(@pagy, next_page || @pagy.page), class: "btn", aria: { disabled: next_page.nil? }.compact_blank do %>
         <%= rails_pulse_icon 'chevron-right', width: '16', height: '16' %>
         <span class="sr-only">Go to next page</span>
       <% end %>
-      <%= link_to pagy_url_for(@pagy, @pagy.last), class: "btn", aria: { disabled: @pagy.next.nil? }.compact_blank do %>
+      <%= link_to pagy_page_url(@pagy, @pagy.last), class: "btn", aria: { disabled: next_page.nil? }.compact_blank do %>
         <%= rails_pulse_icon 'chevrons-right', width: '16', height: '16' %>
         <span class="sr-only">Go to last page</span>
       <% end %>

data/app/views/rails_pulse/csp_test/show.html.erb

@@ -171,7 +171,7 @@
       <div class="card">
         <div class="p-4">
           <h2 class="text-lg font-semibold mb-3">Chart Component Test</h2>
-          <p class="text-subtle mb-4">Testing rails_charts CSP compliance:</p>
+          <p class="text-subtle mb-4">Testing chart CSP compliance:</p>
 
           <div id="chart-container" class="bg-shade border border-main rounded-md p-4 text-center">
             <span class="text-subtle">Chart components would render here in a real dashboard</span>

data/app/views/rails_pulse/dashboard/charts/_bar_chart.html.erb

@@ -1 +1 @@
-<%= bar_chart @component_data, height: "100%", options: bar_chart_options(units: "ms") %>
+<%= render_stimulus_chart @component_data, type: 'bar', height: "100%", options: bar_chart_options(units: "ms") %>

data/app/views/rails_pulse/dashboard/index.html.erb

@@ -1,3 +1,7 @@
+<div class="flex justify-end mb-1">
+  <%= render 'rails_pulse/components/active_filters' %>
+</div>
+
 <div class="row">
   <%= render 'rails_pulse/components/metric_card', { class: "grid-item block", data: @average_query_times_metric_card } %>
   <%= render 'rails_pulse/components/metric_card', { class: "grid-item block", data: @percentile_response_times_metric_card } %>
@@ -16,9 +20,9 @@
     } do %>
       <% if @average_response_time_chart_data.present? %>
         <div class="chart-container chart-container--slim">
-          <%= bar_chart(
+          <%= render_stimulus_chart(
             @average_response_time_chart_data,
-            code: false,
+            type: 'bar',
             id: "dashboard_average_response_time_chart",
             height: "100%",
             options: bar_chart_options(
@@ -40,8 +44,9 @@
     } do %>
       <% if @p95_response_time_chart_data.present? %>
         <div class="chart-container chart-container--slim">
-          <%= bar_chart(
+          <%= render_stimulus_chart(
             @p95_response_time_chart_data,
+            type: 'bar',
             code: false,
             id: "dashboard_p95_response_time_chart",
             height: "100%",

data/app/views/rails_pulse/queries/index.html.erb

@@ -1,4 +1,4 @@
-<%= render 'rails_pulse/components/page_header' %>
+<%= render 'rails_pulse/components/page_header', show_active_filters: true %>
 
 <% unless turbo_frame_request? %>
   <div class="row">
@@ -32,8 +32,9 @@
           class="chart-container chart-container--slim"
           data-rails-pulse--index-target="chart"
         >
-          <%= bar_chart(
+          <%= render_stimulus_chart(
             @chart_data,
+            type: 'bar',
             code: false,
             id: "average_query_times_chart",
             height: "100%",

data/app/views/rails_pulse/queries/show.html.erb

@@ -32,8 +32,9 @@
           class="chart-container chart-container--slim"
           data-rails-pulse--index-target="chart"
         >
-          <%= bar_chart(
+          <%= render_stimulus_chart(
             @chart_data,
+            type: 'bar',
             code: false,
             id: "query_responses_chart",
             height: "100%",

data/app/views/rails_pulse/requests/index.html.erb

@@ -1,4 +1,4 @@
-<%= render 'rails_pulse/components/page_header' %>
+<%= render 'rails_pulse/components/page_header', show_active_filters: true %>
 
 <% unless turbo_frame_request? %>
   <div class="row">

data/app/views/rails_pulse/routes/index.html.erb

@@ -1,4 +1,4 @@
-<%= render 'rails_pulse/components/page_header' %>
+<%= render 'rails_pulse/components/page_header', show_active_filters: true %>
 
 <% unless turbo_frame_request? %>
   <div class="row">
@@ -34,8 +34,9 @@
           class="chart-container chart-container--slim"
           data-rails-pulse--index-target="chart"
         >
-          <%= bar_chart(
+          <%= render_stimulus_chart(
             @chart_data,
+            type: 'bar',
             code: false,
             id: "average_response_times_chart",
             height: "100%",

data/app/views/rails_pulse/routes/show.html.erb

@@ -35,8 +35,9 @@
             class="chart-container chart-container--slim"
             data-rails-pulse--index-target="chart"
           >
-            <%= bar_chart(
+            <%= render_stimulus_chart(
               @chart_data,
+            type: 'bar',
               code: false,
               id: "route_responses_chart",
               height: "100%",

data/app/views/rails_pulse/tags/_tag_manager.html.erb

@@ -11,7 +11,7 @@
 >
   <div class="tag-list">
     <% current_tags.each do |tag| %>
-      <span class="tag">
+      <span class="badge badge--secondary font-normal">
         <%= tag %>
         <%= button_to remove_tag_path(taggable_type, taggable.id, tag: tag),
               method: :delete,
@@ -29,7 +29,7 @@
         <button
           type="button"
           id="tag_menu_button_<%= taggable_type %>_<%= taggable.id %>"
-          class="btn btn--sm tag-add-button"
+          class="badge badge--positive tag-add-button font-normal"
           data-rails-pulse--popover-target="button"
           data-action="rails-pulse--popover#toggle"
           aria-haspopup="true"

data/config/importmap.rb

@@ -1,6 +1,6 @@
 pin "application", to: "rails_pulse/application.js"
 
-# echarts is a dependency of the rails_charts gem
+# echarts is used for chart rendering
 pin "echarts", to: "echarts.min.js"
 # pin "echarts/theme/inspired", to: "echarts/theme/inspired.js"
 pin "rails_pulse/theme", to: "rails_pulse/theme.js"