diff --git a/app/assets/stylesheets/legislation_process.scss b/app/assets/stylesheets/legislation_process.scss
index 3cd63e889..c605af7e7 100644
--- a/app/assets/stylesheets/legislation_process.scss
+++ b/app/assets/stylesheets/legislation_process.scss
@@ -191,12 +191,10 @@
margin-bottom: 0;
text-transform: uppercase;
}
- }
- h4 a {
- @include brand-color;
-
- &:hover {
+ a {
+ @include brand-color;
+ @include header-font-size(h4);
text-decoration: none;
}
}
@@ -232,6 +230,7 @@
}
.quiz-question {
+ @include header-font-size(h2);
margin-bottom: $line-height;
}
diff --git a/app/models/legislation/question.rb b/app/models/legislation/question.rb
index 89515d943..c0ea91527 100644
--- a/app/models/legislation/question.rb
+++ b/app/models/legislation/question.rb
@@ -4,6 +4,7 @@ class Legislation::Question < ApplicationRecord
include Notifiable
translates :title, touch: true
+ translates :description, touch: true
include Globalizable
belongs_to :author, -> { with_hidden }, class_name: "User", inverse_of: :legislation_questions
diff --git a/app/views/admin/legislation/questions/_form.html.erb b/app/views/admin/legislation/questions/_form.html.erb
index e5338389b..d0a0e646d 100644
--- a/app/views/admin/legislation/questions/_form.html.erb
+++ b/app/views/admin/legislation/questions/_form.html.erb
@@ -7,7 +7,11 @@
<%= f.translatable_fields do |translations_form| %>
- <%= translations_form.text_area :title, rows: 5 %>
+ <%= translations_form.text_field :title %>
+
+
+
+ <%= translations_form.text_area :description, class: "html-area admin" %>
<% end %>
diff --git a/app/views/legislation/questions/show.html.erb b/app/views/legislation/questions/show.html.erb
index dc4939f29..7663fab74 100644
--- a/app/views/legislation/questions/show.html.erb
+++ b/app/views/legislation/questions/show.html.erb
@@ -5,7 +5,7 @@
-
<%= @question.title %>
+
<%= @question.title %>
+ <%= AdminWYSIWYGSanitizer.new.sanitize(@question.description) %>
<%= render "answer_form", process: @process, question: @question, answer: @answer %>
@@ -36,7 +37,7 @@
diff --git a/db/migrate/20220924224924_add_description_to_legislation_question_translations.rb b/db/migrate/20220924224924_add_description_to_legislation_question_translations.rb
new file mode 100644
index 000000000..d2321b70b
--- /dev/null
+++ b/db/migrate/20220924224924_add_description_to_legislation_question_translations.rb
@@ -0,0 +1,5 @@
+class AddDescriptionToLegislationQuestionTranslations < ActiveRecord::Migration[6.0]
+ def change
+ add_column :legislation_question_translations, :description, :text
+ end
+end
diff --git a/db/schema.rb b/db/schema.rb
index 7b3a7349d..802fe82b0 100644
--- a/db/schema.rb
+++ b/db/schema.rb
@@ -861,6 +861,7 @@ ActiveRecord::Schema.define(version: 2023_02_06_141152) do
t.datetime "updated_at", null: false
t.text "title"
t.datetime "hidden_at"
+ t.text "description"
t.index ["hidden_at"], name: "index_legislation_question_translations_on_hidden_at"
t.index ["legislation_question_id"], name: "index_d34cc1e1fe6d5162210c41ce56533c5afabcdbd3"
t.index ["locale"], name: "index_legislation_question_translations_on_locale"
diff --git a/spec/system/admin/legislation/questions_spec.rb b/spec/system/admin/legislation/questions_spec.rb
index 67601cb59..0fbe2a9ab 100644
--- a/spec/system/admin/legislation/questions_spec.rb
+++ b/spec/system/admin/legislation/questions_spec.rb
@@ -32,6 +32,7 @@ describe "Admin legislation questions", :admin do
click_link "Create question"
fill_in "Question", with: "Question 3"
+ fill_in_ckeditor "Description", with: "A little description about question 3"
click_button "Create question"
expect(page).to have_content "Question 3"
@@ -40,7 +41,7 @@ describe "Admin legislation questions", :admin do
context "Update" do
scenario "Valid legislation question" do
- create(:legislation_question, title: "Question 2", process: process)
+ create(:legislation_question, title: "Question 2", description: "Description 2", process: process)
visit admin_root_path
@@ -57,9 +58,11 @@ describe "Admin legislation questions", :admin do
click_link "Question 2"
fill_in "Question", with: "Question 2b"
+ fill_in_ckeditor "Description", with: "Description 2b"
click_button "Save changes"
expect(page).to have_content "Question 2b"
+ expect(page).to have_ckeditor "Description", with: "Description 2b"
end
end
diff --git a/spec/system/legislation/questions_spec.rb b/spec/system/legislation/questions_spec.rb
index b217ad7f4..41570c6a6 100644
--- a/spec/system/legislation/questions_spec.rb
+++ b/spec/system/legislation/questions_spec.rb
@@ -9,9 +9,9 @@ describe "Legislation" do
end
before do
- create(:legislation_question, process: process, title: "Question 1")
- create(:legislation_question, process: process, title: "Question 2")
- create(:legislation_question, process: process, title: "Question 3")
+ create(:legislation_question, process: process, title: "Question 1", description: "Description 1")
+ create(:legislation_question, process: process, title: "Question 2", description: "Description 2")
+ create(:legislation_question, process: process, title: "Question 3", description: "Description 3")
end
scenario "shows question list" do
@@ -24,16 +24,19 @@ describe "Legislation" do
click_link "Question 1"
expect(page).to have_content("Question 1")
+ expect(page).to have_content("Description 1")
expect(page).to have_content("NEXT QUESTION")
click_link "Next question"
expect(page).to have_content("Question 2")
+ expect(page).to have_content("Description 2")
expect(page).to have_content("NEXT QUESTION")
click_link "Next question"
expect(page).to have_content("Question 3")
+ expect(page).to have_content("Description 3")
expect(page).not_to have_content("NEXT QUESTION")
end
@@ -41,6 +44,7 @@ describe "Legislation" do
visit legislation_process_question_path(process, process.questions.first)
expect(page).to have_content("Question 1")
+ expect(page).to have_content("Description 1")
expect(page).to have_content("Open answers (0)")
end
@@ -48,16 +52,19 @@ describe "Legislation" do
visit legislation_process_question_path(process, process.questions.first)
expect(page).to have_content("Question 1")
+ expect(page).to have_content("Description 1")
expect(page).to have_content("NEXT QUESTION")
click_link "Next question"
expect(page).to have_content("Question 2")
+ expect(page).to have_content("Description 2")
expect(page).to have_content("NEXT QUESTION")
click_link "Next question"
expect(page).to have_content("Question 3")
+ expect(page).to have_content("Description 3")
expect(page).not_to have_content("NEXT QUESTION")
end