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

[default] Lack of modal backdrop #2319

Closed
vue-kacper opened this issue Jan 31, 2019 · 5 comments
Closed

[default] Lack of modal backdrop #2319

vue-kacper opened this issue Jan 31, 2019 · 5 comments
Labels
bug Bug reports good first issue Tasks that requires just basic understanding of Vue Storefront vs-hackathon Tasks for the Hackathon
Milestone

Comments

@vue-kacper
Copy link
Contributor

Current behavior

Lack of modal backdrop entering "My account" from sidebar menu

Expected behavior

Modal backdrop should be visible

Steps to reproduce the issue

Visit: https://demo.storefrontcloud.io as a guest
Click on the menu
Click "My account"

Additional information

image

Should be:

image

@vue-kacper vue-kacper changed the title Lack of modal backdrop [default] Lack of modal backdrop Jan 31, 2019
@filrak
Copy link
Collaborator

filrak commented Jan 31, 2019

Tip - there is an action for showing overlay in ui module

@filrak filrak added bug Bug reports good first issue Tasks that requires just basic understanding of Vue Storefront vs-hackathon Tasks for the Hackathon labels Jan 31, 2019
giuliachiola referenced this issue in giuliachiola/vue-storefront Feb 1, 2019
filrak referenced this issue Feb 1, 2019
ref. DivanteLtd#2319 modal backdrop
@lorenaramonda
Copy link
Collaborator

lorenaramonda commented Feb 1, 2019

I'm sorry to tell 😬 that the fix provided seems to "work" only when not logged in but when user are logged in, the submenu links all break by opening the login modal and consequently breaking the toggle of overlay and closing of sidebar.

This problem rise up because of this behavior:

  • clicking on the my account link triggers the modal open and sets state.overlay to true (although it still doesn't show the modal to frontend)
  • then the click propagation also triggers the closing of the menu by setting again the state.overlay to false but still not showing anything to frontend
  • when state.overlay eventually has his own fixed state, modal opens and menu closes but at that point overlay already "knows" that must not show itself

We should find a place, possibly the right one, where checking if a modal is triggered and a overlay is already shown, to not hide it. This should prevent any other similar bug.

@pkarw
Copy link
Collaborator

pkarw commented Feb 2, 2019

@qiqqq @filrak can You guys Please help in here? We should have this fixed before 1.8

@francescosalvi
Copy link

@pkarw I actually have developed a different solution during the hackathon, which I didn't push since is not very clean in my opinion (and there was also the other solution pending); I will when I get back at the pc so you guys can check if it's valid

@pkarw
Copy link
Collaborator

pkarw commented Feb 2, 2019

Perfect :) Thank You @francescosalvi

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug reports good first issue Tasks that requires just basic understanding of Vue Storefront vs-hackathon Tasks for the Hackathon
Projects
None yet
Development

No branches or pull requests

5 participants