Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add description to Legislation questions #4997

Merged
merged 3 commits into from
Feb 23, 2023
Merged

Conversation

decabeza
Copy link
Collaborator

@decabeza decabeza commented Sep 27, 2022

References

This PR resolves #4907.

Objectives

  • Replace Legislation question title textarea to text field.
  • Add description to Legislation questions.

Visual Changes

Legislation question admin form (BEFORE)

1_before

Legislation question admin form (AFTER)

1_after

Legislation question admin index

2_questions

Legislation question user's view

3

@decabeza decabeza self-assigned this Sep 27, 2022
@taitus taitus self-assigned this Feb 20, 2023
@taitus taitus force-pushed the legislation_question branch 2 times, most recently from 509a0de to 8f56b75 Compare February 20, 2023 14:55
@taitus taitus force-pushed the legislation_question branch from 8f56b75 to f1e5a73 Compare February 20, 2023 18:04
@taitus taitus requested a review from javierm February 20, 2023 18:15
@taitus taitus force-pushed the legislation_question branch from f1e5a73 to 3d7f808 Compare February 21, 2023 07:02
@taitus taitus changed the base branch from master to fix-specs February 21, 2023 09:50
@taitus taitus force-pushed the legislation_question branch from 3d7f808 to 9235a15 Compare February 21, 2023 09:53
Base automatically changed from fix-specs to master February 21, 2023 19:00
@javierm javierm force-pushed the legislation_question branch from 9235a15 to 8000277 Compare February 21, 2023 23:53
@taitus taitus requested a review from javierm February 22, 2023 08:02
@taitus taitus force-pushed the legislation_question branch from 3aac392 to 35c4bd8 Compare February 22, 2023 08:09
@javierm javierm self-assigned this Feb 22, 2023
@taitus taitus force-pushed the legislation_question branch from 35c4bd8 to 30f24db Compare February 22, 2023 17:01
@taitus taitus force-pushed the legislation_question branch 2 times, most recently from 5aa6033 to b7ee935 Compare February 22, 2023 18:12
Since we're now allowing <h2> and <h3> tags in the description, we
remove the broken <h3> tag from quiz-question class. Since its content
is also in the <title> element, and in order to be consistent with other
show actions in the public area, we change this tag to <h1>.

We add <h2> tag in the "Share" heading and we replace the broken <h4>
tag above adding new styles.
@taitus taitus force-pushed the legislation_question branch from b7ee935 to d96a917 Compare February 22, 2023 18:19
@taitus taitus merged commit a26f05f into master Feb 23, 2023
@taitus taitus deleted the legislation_question branch February 23, 2023 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Legislation debates can't be formatted
3 participants