Adding background overlay for flag popup

Created on 2 February 2023, almost 2 years ago
Updated 19 July 2023, over 1 year ago

Problem/Motivation

when user clicks on flag anonymous text, it opens a popup without background overlay, and the popup merges with the whole UI

Steps to reproduce

Proposed resolution

Adding z-index and widget-overlay class to popup fixes this issue.

Remaining tasks

User interface changes

API changes

Data model changes

🐛 Bug report
Status

Needs review

Version

1.0

Component

Code

Created by

🇮🇳India TanujJain-TJ

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @TanujJain-TJ
  • 🇮🇳India TanujJain-TJ

    adding patch to fix this issue.

  • Status changed to Needs review almost 2 years ago
  • Issue was unassigned.
  • Status changed to Needs work over 1 year ago
  • 🇩🇪Germany Anybody Porta Westfalica

    @tanuj. could you please reroll this as MR? I'm not yet sure it's the proper fix.

  • First commit to issue fork.
  • @bharath-kondeti opened merge request.
  • Status changed to Needs review over 1 year ago
  • 🇮🇳India bharath-kondeti Hyderabad

    @AnyBody, opened an MR. Can you please review.

  • 🇩🇪Germany Anybody Porta Westfalica

    Thank you very much - could someone prepare a before / after screenshot perhaps or review this?

  • 🇮🇳India mayursolaskar

    I have attempted to reproduce the reported issue, but I am unable to do so. When I click on the flag text, it opens a popup with a background overlay.please check Screenshot for reference.
    Here are the steps I followed for testing:-
    Installed and enabled the flag_anon module, ensuring that the required dependency (flag module) was also installed.
    Configured the necessary settings for anonymous users as specified in the module.
    Accessed the page as an anonymous user and clicked on the flag text,(Login or Register link).
    As a result, a popup with a background overlay was displayed.
    Thank You

Production build 0.71.5 2024