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

USH-4329: Make anchored submit button sticky #34490

Merged
merged 12 commits into from
May 1, 2024

Conversation

MartinRiese
Copy link
Contributor

@MartinRiese MartinRiese commented Apr 23, 2024

Product Description

Making submit button sticky instead of fixed to the bottom of the page.

2024-04-23_09-17-22.mp4

Technical Summary

https://dimagi.atlassian.net/browse/USH-4329

Feature Flag

USH: Keep submit button anchored at the bottom of screen for forms in Web Apps

Safety Assurance

Safety story

Automated test coverage

QA Plan

Rollback instructions

  • This PR can be reverted after deploy with no further considerations

Labels & Review

  • Risk label is set correctly
  • The set of people pinged as reviewers is appropriate for the level of risk of the change

@dimagimon dimagimon added the Risk: Medium Change affects files that have been flagged as medium risk. label Apr 23, 2024
@MartinRiese MartinRiese marked this pull request as ready for review April 23, 2024 18:39
@nospame
Copy link
Contributor

nospame commented Apr 24, 2024

Seems like this needs build_bootstrap5_diffs run for test failures, but looks good.

* update bootstrap diff
@MartinRiese MartinRiese force-pushed the riese/style-anchored-submit branch from 115bd78 to 354f8a8 Compare April 24, 2024 18:40
@MartinRiese MartinRiese requested review from Jtang-1 and nospame April 25, 2024 13:00
@orangejenny
Copy link
Contributor

confirmed B5 changes look good

@MartinRiese MartinRiese merged commit 8a0b161 into master May 1, 2024
12 checks passed
@MartinRiese MartinRiese deleted the riese/style-anchored-submit branch May 1, 2024 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Risk: Medium Change affects files that have been flagged as medium risk.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants