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

5307: Fixed confirmation message styling #1876

Merged
merged 1 commit into from May 3, 2022

Conversation

arniloftsson
Copy link
Contributor

Link to issue

https://platform.dandigbib.org/issues/5307

Description

The problem was that the form was still there which made the confirmation page extra long and scrolled down to the bottom. Fixed by not displaying the form on confirmation page.

Screenshot of the result

image

Checklist

  • My complies with our coding guidelines.
  • My code passes our static analysis suite. If not then I have added a comment explaining why this change should be exempt from the code standards and process.
  • My code passes our continuous integration process. If not then I have added a comment explaining why this change should be exempt from the code standards and process.

Additional comments or questions

If you have any further comments or questions for the reviewer them please add them here.

@JrgenGNielsen
Copy link
Contributor

reviewed and approved

@kasperg kasperg merged commit 71d4e4a into ding2:master May 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants