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

Uppy DashboardModal white foreground. #1571

Closed
atifaziz2 opened this issue May 20, 2019 · 8 comments
Closed

Uppy DashboardModal white foreground. #1571

atifaziz2 opened this issue May 20, 2019 · 8 comments

Comments

@atifaziz2
Copy link

I am wondering everything was working fine, but suddenly it makes problem, DashboardModal is showing with white foreground.

screenshot is attached.

white_dashboardmodal

@atifaziz2 atifaziz2 added the Not Accepted We cannot accept this issue, please read https://uppy.io/support label May 20, 2019
@arturi
Copy link
Contributor

arturi commented May 20, 2019

Hi! Does the problem persist when you reload the page multiple times? What browser/OS are you using? And if possible, could you provide the code you are using to initialize Uppy?

@atifaziz2
Copy link
Author

atifaziz2 commented May 20, 2019

I am using chrome browser on Linux.
The problem is when i add

href="https://transloadit.edgly.net/releases/uppy/v1.0.0/uppy.min.css" rel="stylesheet"

into head tag, while it's works fine with old version

href="https://transloadit.edgly.net/releases/uppy/v0.29.1/dist/uppy.min.css" rel="stylesheet"

@arturi
Copy link
Contributor

arturi commented May 20, 2019

When using CSS from v1.0.0, are you using matching version of javascript though, https://transloadit.edgly.net/releases/uppy/v1.0.0/uppy.min.js?

@goto-bus-stop
Copy link
Contributor

goto-bus-stop commented May 20, 2019

Perhaps some of your site's CSS is conflicting with Uppy's. We removed !important from our style rules to make it possible to change Uppy's CSS some time ago. Could you check in the inspector if something is overriding the styles?

e; ok, nvm 😄

@atifaziz2
Copy link
Author

When using CSS from v1.0.0, are you using matching version of javascript though, https://transloadit.edgly.net/releases/uppy/v1.0.0/uppy.min.js?

@arturi you are right, actually i am using 0.29.1.
Thanks for clarify.

@arturi
Copy link
Contributor

arturi commented May 20, 2019

So it works for you now, @atifaziz2 atifaziz2?

Because I suspect the issue is with recent addition of uppy-Dashboard--isInnerWrapVisible class in #1383, without which the Dashboard stays at opacity 0:

.uppy-Dashboard-innerWrap {
display: flex;
flex-direction: column;
height: 100%;
overflow: hidden;
position: relative;
border-radius: 5px;
opacity: 0;
}
.uppy-Dashboard--isInnerWrapVisible .uppy-Dashboard-innerWrap{
opacity: 1;
}

@arturi arturi added Dashboard and removed Not Accepted We cannot accept this issue, please read https://uppy.io/support labels May 20, 2019
@atifaziz2
Copy link
Author

So it works for you now, @atifaziz2 atifaziz2?

Because I suspect the issue is with recent addition of uppy-Dashboard--isInnerWrapVisible class in #1383, without which the Dashboard stays at opacity 0:

.uppy-Dashboard-innerWrap {
display: flex;
flex-direction: column;
height: 100%;
overflow: hidden;
position: relative;
border-radius: 5px;
opacity: 0;
}
.uppy-Dashboard--isInnerWrapVisible .uppy-Dashboard-innerWrap{
opacity: 1;
}

Yes @arturi , it works fine.

@arturi
Copy link
Contributor

arturi commented May 20, 2019

Great, closing then!

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

No branches or pull requests

3 participants