-
Notifications
You must be signed in to change notification settings - Fork 23
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
fix: re-order change and analysis modal titles and add overflow #1405
fix: re-order change and analysis modal titles and add overflow #1405
Conversation
✅ Deploy Preview for goofy-euclid-75956c ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for clerk-saas-ui ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for flanksource-demo-stable ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@mainawycliffe can you check the catalogs panel - we seem to have lost the overflow ellipses there |
@moshloop which link is this? I am looking for one to check if it's fixed, can't find one with long catalog names in the panel. |
c11c82c
to
e163c18
Compare
@mainawycliffe It looks like we also have a modal overflow issue if you click on one of the AttachVolume events - can we not force all modal bodies globally to scroll overflow ? |
@moshloop let me re-design the modal for this, it should be straight forward to achieve that for the modals. |
Overflow on last item now shows ellipses Closes #1379
e163c18
to
56a624a
Compare
Overflow on last item now shows ellipses
Closes #1379