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

Fix style rule of error message when enterprise does not have shipping or payment method activated #12362

Open
1 of 2 tasks
RachL opened this issue Apr 11, 2024 · 11 comments
Open
1 of 2 tasks
Assignees
Labels
bug-s3 The bug is stopping a critical or non-critical feature but there is a usable workaround. good first issue papercut Labels papercuts after they've moved from the "Papercuts prioritized" column

Comments

@RachL
Copy link
Contributor

RachL commented Apr 11, 2024

Description

When an enterprise has no shipping or payment methods activated, an error message is displayed on all pages. It is overlapping with the

  • bottom saving banner, preventing the user to fix the error message easily. (e.g. at /admin/enterprises/*/edit)
  • bottom of the new bulk edit products screen, preventing interaction (/admin/products with admin_style_v3 feature toggle)

The goal here is to fix the style rule so that you can at least click in the space underneath the error message

Steps to Reproduce

  1. Remove shipping and payment methods from your enterprise
  2. See the error message
  3. try to re-create the methods, see it overlap with the bottom of the screen

Animated Gif/Screenshot

In bulk edit products, you cannot click anywhere in the purple area:

image

Loom video: https://www.loom.com/share/2fb04215bdcb424f923a52606ef7c088?sid=0d6e542f-08eb-4d50-ae11-ad755c208615

Severity

bug-s3: a feature is broken but there is a workaround

Your Environment

  • Version used: v4.4.38
  • Browser name and version: Firefox and chrome
  • Operating System and version (desktop or mobile): both
@RachL RachL added good first issue bug-s3 The bug is stopping a critical or non-critical feature but there is a usable workaround. papercut Labels papercuts after they've moved from the "Papercuts prioritized" column labels Apr 11, 2024
@RachL RachL changed the title Fix style rule of error message when enterprise doe snot have shipping or payment method activated Fix style rule of error message when enterprise does not have shipping or payment method activated Apr 11, 2024
@MrBowmanXD
Copy link
Contributor

MrBowmanXD commented May 23, 2024

https://youtu.be/p9iBHp-C0Fc

It seems the error message is on top of the screen. Well stylized and permits interaction with the bottom panel.

@RachL
Copy link
Contributor Author

RachL commented May 23, 2024

@MrBowmanXD sorry I forgot an important information when writting the issue: you need to turn feature toggle "admin_style_v3" ON to see the problem.

You can fin toggles as superadmin on /admin/feature-toggle/features more info at https://github.com/openfoodfoundation/openfoodnetwork/wiki/Feature-toggles

@MrBowmanXD
Copy link
Contributor

will try to replicate.

@MrBowmanXD
Copy link
Contributor

Image

The banner is stylized in a way that permits to see the update button (does not overlap) but you can't click on the update button without dismissing the error message, seems like a part of the design choice.

@RachL
Copy link
Contributor Author

RachL commented May 23, 2024

@tschumilas or @dacook can you still replicate? I can't on my end, but I never could, the screenshot in the issue is actually yours David :) maybe the issue is missing an important step.

@dacook
Copy link
Member

dacook commented May 26, 2024

Hi @MrBowmanXD , thanks for looking into this. I'm sorry, the issue was out of date. I've updated it now to show that there were two problems. The one you checked is solved already, but the remaining issue is at the:

bottom of the new bulk edit products screen, preventing interaction (/admin/products with admin_style_v3 feature toggle)

@MrBowmanXD
Copy link
Contributor

Can you replicate in a video or image please? Did not understand the text:

| bottom of the new bulk edit products screen, preventing interaction (/admin/products with admin_style_v3 feature toggle)

@MrBowmanXD
Copy link
Contributor

MrBowmanXD commented May 27, 2024

Wait, so should div called flash container be behind the content in order to let user click on things? (possibly by lowering z-index) I have to check this out in the code editor.

@dacook
Copy link
Member

dacook commented May 28, 2024

It sounds like you're on the right track, yes. I've added a Loom video to the description of this issue, I hope that helps.

@MrBowmanXD
Copy link
Contributor

Thanks for the video, will get on this issue soon.

@dacook
Copy link
Member

dacook commented Jun 6, 2024

Hi @MrBowmanXD , just wondering if you'll be able to look into this soon?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-s3 The bug is stopping a critical or non-critical feature but there is a usable workaround. good first issue papercut Labels papercuts after they've moved from the "Papercuts prioritized" column
Projects
Status: In Progress ⚙
Status: In progress
Development

No branches or pull requests

3 participants