Skip to content

fix: OPTIC-883: Export data modal footer not visible when scrollable #6085

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

Merged
merged 1 commit into from
Jul 12, 2024

Conversation

bmartel
Copy link
Contributor

@bmartel bmartel commented Jul 11, 2024

App redesign had changed a couple of the overflow values on modals, which broke the ability to provide a position sticky footer. This made it difficult to present modal forms with scrolling content and have the form submit button visible at all times.

Before
image

After
image

@github-actions github-actions bot added the fix label Jul 11, 2024
Copy link

netlify bot commented Jul 11, 2024

Deploy Preview for heartex-docs canceled.

Name Link
🔨 Latest commit c6bf228
🔍 Latest deploy log https://app.netlify.com/sites/heartex-docs/deploys/66902e7d650fae00081e21e5

Copy link

netlify bot commented Jul 11, 2024

Deploy Preview for label-studio-docs-new-theme canceled.

Name Link
🔨 Latest commit c6bf228
🔍 Latest deploy log https://app.netlify.com/sites/label-studio-docs-new-theme/deploys/66902e7d420e070007125a9a

Copy link
Contributor

@yyassi-heartex yyassi-heartex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

negative margins are weird - but given its been around for a while we should take another look at that later

@hlomzik
Copy link
Collaborator

hlomzik commented Jul 11, 2024

negative margins are weird - but given its been around for a while we should take another look at that later

@yyassi-heartex it's a popular workaround with outer paddings you can't control. it would be better without it with better layout for sure, but I think they are just okay

@bmartel bmartel enabled auto-merge (squash) July 12, 2024 13:41
@bmartel bmartel merged commit 0026d23 into develop Jul 12, 2024
31 checks passed
bmartel added a commit that referenced this pull request Jul 12, 2024
…6085)

App redesign had changed a couple of the overflow values on modals,
which broke the ability to provide a position sticky footer. This made
it difficult to present modal forms with scrolling content and have the
form submit button visible at all times.

_Before_

![image](https://github.com/user-attachments/assets/839b77f9-8f3e-467d-adb2-5c8ac3727ade)

_After_

![image](https://github.com/user-attachments/assets/cb1dd835-a537-43a7-8951-c1dd896ae615)
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.

5 participants