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

[Modal] Fullscreen modal stuck to left edge of page #6587

Closed
GammaGames opened this issue Sep 18, 2018 · 7 comments
Closed

[Modal] Fullscreen modal stuck to left edge of page #6587

GammaGames opened this issue Sep 18, 2018 · 7 comments

Comments

@GammaGames
Copy link
Contributor

Steps

  1. Create a fullscreen modal in html
  2. Call the modal function on it and show

Expected Result

Modal should be centered

Actual Result

Modal is stuck to left edge of page

Version

2.4.0

Testcase

https://jsfiddle.net/Ljcyd5ek/

It can be fixed by adding the following css into the page to override it:

.ui.fullscreen.modal {
  left: 2.5% !important;
}
@pfcodes
Copy link

pfcodes commented Sep 20, 2018

+1

@y0hami y0hami added this to the 2.4.1 milestone Sep 20, 2018
@pfcodes
Copy link

pfcodes commented Sep 20, 2018

@GammaGames The CSS to override isn't working for me. I'm using the SUI CSS only version and even tried editing the source stylesheet file to no avail. Any other quick dirty fix?

Edit: I resolved my issue with .ui.fullscreen.scrolling.modal

@GammaGames
Copy link
Contributor Author

Yeah, the .ui.fullscreen.scrolling.modal was more specific, so it was overriding the base .ui.fullscreen.modal rules. If you want to read more, here's a short explanation.

Also, in the future just add a thumb to the thread instead of a +1 comment. Everyone subscribed (with this thread it's not many) gets a notification whenever someone comments, and GitHub has tools to sort by the most reactions to see which issues have the most support.

@jlukic
Copy link
Member

jlukic commented Sep 24, 2018

I'll fix this.

@jlukic
Copy link
Member

jlukic commented Oct 13, 2018

I've fixed this for patch today

No left offset is necessary with flex or legacy modals. With legacy it will be calculated in javascript. In flex, it's handled with flex container.

@jlukic jlukic modified the milestones: 2.4.2, 2.4.1 Oct 13, 2018
@jlukic jlukic closed this as completed Oct 13, 2018
@SaphuA
Copy link

SaphuA commented Dec 21, 2018

@jlukic

This does not appear to be fixed in v2.4.2??

See -> https://codepen.io/anon/pen/aPJLya

@lubber-de

This comment was marked as spam.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants