geta-notfoundhandler icon indicating copy to clipboard operation
geta-notfoundhandler copied to clipboard

Show modal in same position (height) as trigger of modal

Open ellinge opened this issue 2 years ago • 4 comments

Fixes #72

Had to do a bit of a workaround since the table / modal is within an iframe so not easily fixed by only css.

Either way the modal is now positioned based on the offset height of the button which triggered it.

image

ellinge avatar Feb 10 '23 10:02 ellinge

Not sure what to to with the build failing, tests seems to pass but I think your sonar-configuration is off.

Seems to have to do with the build being processed on my fork where the secret is missing: https://community.sonarsource.com/t/sonarcloud-analysis-fails-on-prs-for-c-project-on-github/34260

https://portal.productboard.com/sonarsource/1-sonarcloud/c/50-sonarcloud-analyzes-external-pull-request

ellinge avatar Feb 10 '23 10:02 ellinge

The build fails for the reason you wrote. Please ignore it. Regarding the changes, the fix should be possible without JS and just with CSS. Also, it would be good to update Bootstrap to the release version (now it uses Beta) before applying the fix - it might fix the issue. I will take a look at it. If none of it will help, then this PR will be accepted.

marisks avatar Feb 17 '23 08:02 marisks

I don’t think it wont be doable with css only because the iframe viewport. It’s if bootstrap has some js to automatically apply it.

Basically this still applies (a quick google): https://stackoverflow.com/questions/8061728/fix-an-element-inside-an-iframe-in-the-center-of-the-screen

ellinge avatar Feb 17 '23 08:02 ellinge

You are right. Bootstrap already sets it in the center, but the issue is that it is in an iframe.

marisks avatar Feb 17 '23 10:02 marisks

The modal will go behind the screen If there are more than 10 referrers in the first entries.

Example: image

There should be a fix for such situations, or better just show it always centered taking into account the current scrolling position.

jevgenijsp avatar Oct 28 '24 14:10 jevgenijsp

Closing this ticket due to creating a copy of this with the additional fix: https://github.com/Geta/geta-notfoundhandler/pull/147

jevgenijsp avatar Nov 05 '24 09:11 jevgenijsp