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

🐛 Inconsistent Modal Display for "See External Data" on Different Screen Sizes #2555

Open
kneerose opened this issue Dec 19, 2024 · 1 comment
Assignees
Labels
🐛 Bug Something isn't working

Comments

@kneerose
Copy link
Contributor

kneerose commented Dec 19, 2024

Area

Other

Domain

dev-sanchogov.tools

Which wallet were you using?

No response

Context

When navigating to a governance action with incorrectly formatted data, clicking "See External Data" triggers a modal popup. On a desktop-sized screen, the modal displays a warning with the title "Be careful":

image

However, resizing the screen to a mobile view changes the modal to display a different title: "External Link Safety":

image

Steps to reproduce

  1. Navigate to any governance action page with data formatted incorrectly.
  2. Click on "See External Data".
  3. Observe the modal with the title "Be careful".
  4. Resize the screen to a mobile view.
  5. Notice the modal title changes to "External Link Safety".

https://www.loom.com/share/d453cc5c386e4ffeb51093a60e310200?sid=21155d0e-7596-4c6b-8fca-906ac659a8f7

Actual behavior

Two different modals are displayed for the same action depending on the screen size.

Expected behavior

The same modal should be displayed regardless of screen size for the same action.

Question

  • Is this the intended behavior, or was it overlooked for smaller screens?

Allure Report

@bosko-m
Copy link
Contributor

bosko-m commented Dec 19, 2024

Great catch @kneerose

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug Something isn't working
Projects
Status: No status
Status: Backlog
Development

No branches or pull requests

3 participants