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

Use div for inspector form wrapped extra questions #3250

Merged
merged 1 commit into from Nov 26, 2020

Conversation

dracos
Copy link
Member

@dracos dracos commented Nov 23, 2020

As extra questions can include limited HTML now, including <p>,
wrapping them in a <p> can lead to display issues.

@dracos dracos requested a review from davea November 23, 2020 12:53
@codecov
Copy link

codecov bot commented Nov 23, 2020

Codecov Report

Merging #3250 (dfe980a) into master (80e3815) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3250   +/-   ##
=======================================
  Coverage   81.37%   81.37%           
=======================================
  Files         309      309           
  Lines       20660    20660           
  Branches     3130     3130           
=======================================
  Hits        16812    16812           
  Misses       2826     2826           
  Partials     1022     1022           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 80e3815...dfe980a. Read the comment docs.

As extra questions can include limited HTML now, including <p>,
wrapping them in a <p> can lead to display issues.
@dracos dracos merged commit dfe980a into master Nov 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants