fix a11y-8.9.1
no p tag when Champ text with no value
This commit is contained in:
parent
1f219de931
commit
447612abdf
3 changed files with 3 additions and 2 deletions
|
@ -1,5 +1,6 @@
|
|||
module StringToHtmlHelper
|
||||
def string_to_html(str, wrapper_tag = 'p')
|
||||
return nil if str.blank?
|
||||
html_formatted = simple_format(str, {}, { wrapper_tag: wrapper_tag })
|
||||
with_links = Anchored::Linker.auto_link(html_formatted, target: '_blank', rel: 'noopener')
|
||||
sanitize(with_links, attributes: ['target', 'rel', 'href'])
|
||||
|
|
|
@ -57,7 +57,7 @@
|
|||
- when TypeDeChamp.type_champs.fetch(:number)
|
||||
= number_with_html_delimiter(c.to_s)
|
||||
- else
|
||||
= format_text_value(c.to_s)
|
||||
= format_text_value(c.to_s) unless c.blank?
|
||||
|
||||
- if c.type_champ != TypeDeChamp.type_champs.fetch(:header_section)
|
||||
%td.updated-at
|
||||
|
|
|
@ -28,7 +28,7 @@ RSpec.describe StringToHtmlHelper, type: :helper do
|
|||
context "with empty decription" do
|
||||
let(:description) { nil }
|
||||
|
||||
it { is_expected.to eq('<p></p>') }
|
||||
it { is_expected.to eq nil }
|
||||
end
|
||||
|
||||
context "with a bad script" do
|
||||
|
|
Loading…
Add table
Reference in a new issue