letter_opener_web 2.0.0.pre.alpha1 → 2.0.0.pre.beta

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '0769c48d5d3d5f4a4353b406d61fc4f38982643c1ceb2a95a2695e1f32e2d299'
-  data.tar.gz: cae0ee6946cdf39ecf07de402902564c773a3d21182625b3131d89031adaefed
+  metadata.gz: 285be7616fcef875c214cf862ce009bad30562c1de5f0cd7ea65348ef9201233
+  data.tar.gz: ad8acc2f43564626b87a5950debe40700f3f492f29a397fb5f8d88f6ae516747
 SHA512:
-  metadata.gz: b26f6bafa83f0d690ca8ce7d7f01fb343836c258216d43f1c7b12c083f09d4bf60ca7466e6400f33f8cf210fa5fe55bfebd10d4f14f6370d10e4256e01cb5fad
-  data.tar.gz: 299ebe6fc4b908b7be0ed2af430fc820e85aa9c6903a815d0258692cfbd80888d1fc91795a319157df9097965ea2e5adfcbd21a47b44dddc3f879edc1e301295
+  metadata.gz: 7355ec8c5c8ec6dbaa7b459475983a6da804d3f35ca30cdcb15d9b1b206877b2a92af5f0e02adc4850cebd1045c0f9ea57399d45cb883c6d357a3f9cf786cc80
+  data.tar.gz: f001b0b9f78aa875f6130bfb0b7443c3647f2b1edff3fc582d44890cbfafb7e0bdc55aa18ae4da30f8e936a6dda5168922414047b984f266fbbaba3a363bc604

data/CHANGELOG.md

@@ -3,7 +3,7 @@
   - Require Rails >= 5.2, run tests against Rails 6.1 [#113](https://github.com/fgrehm/letter_opener_web/pull/113)
   - Upgrade to Bootstrap 4.2 [#113](https://github.com/fgrehm/letter_opener_web/pull/113)
   - Inline CSS and Javascript [#113](https://github.com/fgrehm/letter_opener_web/pull/113)
-  - Add rexml gem into dependency for Ruby 3.0 [#160](https://github.com/fgrehm/letter_opener_web/pull/106)
+  - Add rexml gem into dependency for Ruby 3.0 [#106](https://github.com/fgrehm/letter_opener_web/pull/106)
   - Add routes for Rails API mode [#69](https://github.com/fgrehm/letter_opener_web/pull/69)
   - Prevent name conflict with `Letter` class [#108](https://github.com/fgrehm/letter_opener_web/pull/108)
   - Add Rails' built-in CSRF protection [#111](https://github.com/fgrehm/letter_opener_web/pull/111)

data/app/controllers/letter_opener_web/application_controller.rb

@@ -2,6 +2,6 @@
 
 module LetterOpenerWeb
   class ApplicationController < ActionController::Base
-    protect_from_forgery with: :exception
+    protect_from_forgery with: :exception, unless: -> { Rails.configuration.try(:api_only) }
   end
 end

data/app/controllers/letter_opener_web/letters_controller.rb

@@ -37,7 +37,10 @@ module LetterOpenerWeb
 
     def destroy
       @letter.delete
-      redirect_to routes.letters_path
+      respond_to do |format|
+        format.html { redirect_to routes.letters_path }
+        format.js { render js: "window.location='#{routes.letters_path}'" }
+      end
     end
 
     private

data/app/views/layouts/letter_opener_web/_javascripts.html.erb

@@ -13,8 +13,8 @@ jQuery(function($) {
     $('.letter-opener').on('click', 'a', function() {
         var $this = $(this);
         $('iframe').attr('src', $this.attr('href'));
-        $this.parent().find('.active').removeClass('active');
-        $this.addClass('active');
+        $this.parents('.list-group').find('.active').removeClass('active');
+        $this.parent().addClass('active');
     });
 
     $('.refresh').click(function(e) {

data/app/views/layouts/letter_opener_web/styles/_letters.html.erb

@@ -33,6 +33,14 @@
     padding-left:  5px;
   }
 
+  .message-headers a {
+    color: black;
+  }
+
+  .message-headers.active a {
+    color: white;
+  }
+
   .message-headers dl {
     float: left;
     font-size: 0.8rem;

data/app/views/letter_opener_web/letters/_item.html.erb

@@ -1,5 +1,10 @@
-<% classes = %w(list-group-item list-group-action message-headers p-2) %>
+<% classes = %w(list-group-item list-group-action d-flex justify-content-between align-items-start p-2 message-headers) %>
 <% classes << 'active' if active %>
-<%= link_to(letter_path(letter, style: letter.default_style), class: classes, target: 'mail') do %>
-  <%= letter.headers %>
+<%= content_tag :li, class: classes do %>
+  <%= link_to(letter_path(letter, style: letter.default_style), target: 'mail') do %>
+    <%= letter.headers %>
+  <% end %>
+  <button class="btn btn-danger btn-sm" data-remote="true" data-url="<%= delete_letter_path(letter) %>" data-method="post" data-confirm="Are you sure?">
+    <strong><small>X</small></strong>
+  </button>
 <% end %>

data/app/views/letter_opener_web/letters/index.html.erb

@@ -11,12 +11,12 @@
     </span>
   </h1>
     <div class="letter-opener" data-letters-path="<%= letters_path %>">
-      <div class="list-group pt-2">
+      <ul class="list-group rounded-0 pt-2">
         <% if first_letter = @letters.shift %>
           <%= render partial: "item", locals: { letter: first_letter, active: true } %>
         <% end %>
         <%= render partial: "item", collection: @letters, as: :letter, locals: { active: false } %>
-      </div>
+      </ul>
     </div>
 </div>
 <div class="col right">

data/lib/letter_opener_web/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module LetterOpenerWeb
-  VERSION = '2.0.0-alpha1'
+  VERSION = '2.0.0-beta'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: letter_opener_web
 version: !ruby/object:Gem::Version
-  version: 2.0.0.pre.alpha1
+  version: 2.0.0.pre.beta
 platform: ruby
 authors:
 - Fabio Rehm
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-10-28 00:00:00.000000000 Z
+date: 2021-11-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionmailer