remotestorage / remotestorage-widget

⬡ Connect widget for remoteStorage.js
https://remotestorage.io
28 stars 16 forks source link

Background Modal in Foreground #110

Closed KatFrog closed 4 years ago

KatFrog commented 4 years ago

The background modal shows up in the foreground on both small screens (various Android phones) and on a larger computer screen.

raucao commented 4 years ago

Thanks for the report, @KatFrog.

Could you please link to an example that we can inspect, as this is not a known bug? Thank you.

ProggerPete commented 4 years ago

This is happening for me when viewing in chrome on my android phone. The element causing the issue is:

<div class="remotestorage-widget-modal-backdrop visible" style="display: block;"></div>

I'm working around by overriding the display style so it's always hidden.

div.remotestorage-widget-modal-backdrop {
  display: none !important;
}

This is happening on my test site : https://poker.techforge.com.au/

ProggerPete commented 4 years ago

Created a pull request with a fix : https://github.com/remotestorage/remotestorage-widget/pull/111

raucao commented 4 years ago

@KatFrog To spare you the work of reading through the bug and fix: if you just add a position attribute to your CSS for #remotestorage-widget, the problem should go away. It can be for example relative or fixed or whatever fits your UI.

raucao commented 4 years ago

@KatFrog Just FYI: The fix was just published on npm as version 1.5.2.